Skip to content
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 selectors with data-* style attributes #14

Closed
jakubpawlowicz opened this issue Apr 26, 2010 · 6 comments
Closed

CSS selectors with data-* style attributes #14

jakubpawlowicz opened this issue Apr 26, 2010 · 6 comments

Comments

@jakubpawlowicz
Copy link

less.js have problems parsing the following rule, which is a valid HTML5 tag attribute:
form[data-disabled] {
color:#444;
}

@jamesfoster
Copy link
Contributor

http://github.com/cloudhead/less.js/blob/master/lib/less/parser.js#L663

change key = $(/[a-z]+/g) to key = $(/[a-z-]+/g)

fixed

@cloudhead
Copy link
Member

yah, easy fix : ), will take care of shortly.

@jakubpawlowicz
Copy link
Author

cool. thanks!

@cloudhead
Copy link
Member

fixed.

@zackd
Copy link

zackd commented Aug 23, 2012

this fails: [data-wysihtml5-command] { /* rules */ }

ParseError: Syntax Error on line 42 [data-wysihtml�[7m�[31m�[1m5�[22m-command]

I guess http://github.com/cloudhead/less.js/blob/master/lib/less/parser.js#L663

change key = $(/[a-z]+/g) to key = $(/[a-z-]+/g)

should have been key = $(/[a-z0-9-]+/g) ?

@vjnrv
Copy link

vjnrv commented Oct 1, 2012

I'm experiencing the same issue that @zackd explained.

dhaber added a commit to dhaber/less.js that referenced this issue Jan 22, 2014
…ts (since we don't need to include the test code in the rhino source). Also

sets the stage for sharing lessc code between lessc and rhino less (issue less#14) and provides a hook so lesscss-java can have custom js.
dhaber added a commit to dhaber/less.js that referenced this issue Jan 22, 2014
…ts (since we don't need to include the test code in the rhino source). Also

sets the stage for sharing lessc code between lessc and rhino less (issue less#14) and provides a hook so lesscss-java can have custom js.
dhaber added a commit to dhaber/less.js that referenced this issue Jan 23, 2014
…ts (since we don't need to include the test code in the rhino source). Also

sets the stage for sharing lessc code between lessc and rhino less (issue less#14) and provides a hook so lesscss-java can have custom js.
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants