New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSS parser incorrectly parses "1.em" as "1em" #630

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Apr 6, 2015

We are rewriting:

padding-top: 1.em

to:

padding-top:1em

CSS spec says that the number token is:

num     [0-9]+|[0-9]*\.[0-9]+

Spec CSS parser suggests tokenizing this as:

IDENT(padding-top) : INT(1) DELIM(.) IDENT(em)

not as:

IDENT(padding-top) : DIM(1,em)

Original issue reported on code.google.com by sligocki@google.com on 27 Feb 2013 at 7:26

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

This issue was closed by revision r2570.

Original comment by sligocki@google.com on 1 Mar 2013 at 9:17

  • Changed state: Fixed
@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by sligocki@google.com on 1 Mar 2013 at 9:19

  • Added labels: Milestone-v26, release-note
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment