Valid values for text-decoration need updating

CSSLint / parser-lib

Collection of parsers written in JavaScript

http://www.nczonline.net/

Other

287 stars 82 forks source link

Valid values for text-decoration need updating #219

Closed gouch closed 7 years ago

gouch commented 8 years ago

text-decoration is now a shorthand property and so has several more valid values. csslint incorrectly flags these as errors.

Valid CSS: text-decoration: underline wavy red;
Incorrect csslint error: "Expected end of value but found 'wavy'. (known-properties)"

See documentation

These new text-decoration values are used in the popular normalize.css library.

ideadapt commented 8 years ago

wavy and others are css 3 draft state, which is not covered by parser-lib.

seems like css3 text-decoration is not widely covered yet.

gouch commented 8 years ago

I'm not sure why you think CSSLint ignores draft rules. I see lots of rules in both working draft and candidate recommendation status (e.g. column-rule).

The new text-decoration values are used in normalize.css, which makes its way into a lot of other places (including this very issues page), which is why this error is particularly annoying.

ideadapt commented 8 years ago

I thought so, because I see some of https://github.com/CSSLint/parser-lib/labels/Not%20in%20latest%20official%20spec being in draft state. But your right, Properties.js has a lot of css3 draft things in it :(

Maybe a PR will be accepted. Would you mind to create one?