-
Notifications
You must be signed in to change notification settings - Fork 81
Add missing currentColor keyword #153
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
Conversation
|
Inserting, It seems should be an alphabetical order:D |
|
I'd appreciate this being merged or at least getting some feedback. It's a straightforward bug fix, it passes the tests, and it will resolve issue #590 with CSSLint. |
|
Why was this not merged? The issue is still open and it is mildly annoying. |
|
@gouch: please fetch, rebase and squash the patches. While at it, use a simple single quote instead of the curly ones. |
|
@gouch Please add it at the location in the right alphabetical order. |
|
Please squash the commits into one since they refer to the same thing.
|
|
@XhmikosR Should be one commit now |
|
@gouch: almost there :/ You need to fetch upstream and rebase. |
[Fix CSSLint#131] Add support for the 'fr' unit
* remove obsolete node.js versions * specify `sudo: false` and `fast_finish`
Minor header consistency changes.
Added missing value for the `touch-action` property. Reference: [1](https://developer.mozilla.org/en-US/docs/Web/CSS/touch-action) [2](https://msdn.microsoft.com/en-gb/library/windows/apps/hh767313.aspx)
|
Continued in new pull request: #177 |
This is a valid value that was missing