Skip to content
This repository was archived by the owner on Jan 9, 2021. It is now read-only.

possible ES 5.1 final version #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

possible ES 5.1 final version #4

wants to merge 1 commit into from

Conversation

allenwb
Copy link

@allenwb allenwb commented May 15, 2012

I updated the front page and gave it to Ecma for their feedback. If they are ok with it this can be very close to what gets posted on the Ecma site.

All my changes were made in the actual html file:

cloned the css file so we won't have future name clashes with the es6 version

fixed the following format bugs I found:

Tables used in formatting grammar productions should not have visible grid:

tables in 7.6.1.1, 7.6.1.2, 7.7, 11.13 and corresponding
tables in syntax annex to use "lightweight-table" style instead of
"real-table"

Fixed problem with bullet lists embedded in algorithm steps
in 11.3.1, 11.3.2, 11.4.4, 11.4.5, 11.13.1, 11.13.2

These format fixes probably reflect issues that need to be fixed in the conversion routines.

changed syntax tables in 7.6.1.1, 7.6.1.2, 7.7, 11.13 and corresponding
tables in syntax annex to use "lightweight-table" style instead of
"real-table"  Fixed problem with bullet list embedded in algorithm step
in 11.3.1, 11.3.2, 11.4.4, 11.4.5, 11.13.1, 11.13.2
@@ -2,48 +2,43 @@
<html>
<head>
<title>ECMAScript Language Specification - ECMA-262 Edition 5.1</title>
<link href="es6-draft.css" rel="stylesheet">
<link href="es5.1-final.css" rel="stylesheet">
<link rel="icon" type="image/gif" href="favicon.ico">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double space!!1

@mathiasbynens
Copy link
Contributor

I believe this is fixed in 01a31e8. @jorendorff, if this is correct, you can close this issue.

@jorendorff
Copy link
Owner

Not all the changes are fixed. I have a list and I'll work through them as I have time.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants