Johannes wrote:
On Thu, Feb 19, 2009 at 12:30:34AM +0000, corvid wrote:
Johannes wrote:
On Wed, Feb 11, 2009 at 06:03:13PM +0000, corvid wrote:
I tried font-size: .6in (we still love archaic measurements here) and it didn't work.
The grammar says num [0-9]+|[0-9]*"."[0-9]+
Oh yes. That's Css_next_token() which is not recognizing ".6" as a float...
Attached.
Looks good to me so far. The only thing I'm wooried about is the hack where Css_parse_simple_selector() checks for CSS_TK_FLOAT and interprets them as class selectors. After looking at the CSS syntax I think this is not necessary, as class names must not start with numbers. Can you please have look at the spec too, to confirm this? In that case we could just remove the hack in Css_parse_simple_selector().
I did happen to see: In CSS1, a class name could start with a digit (".55ft"), unless it was a dimension (".55in"). In CSS2, such classes are parsed as unknown dimensions (to allow for future additions of new units). To make ".55ft" a valid class, CSS2 requires the first digit to be escaped (".\35 5ft")