Enable `camelCaseSvgKeywords` option for `value-keyword-case` about stylelint-config-standard HOT 13 OPEN

Make sense. Let's move this into the standard config repo and continue discussing.

Ref:

from stylelint-config-standard.

The link you provided is CSS Color Module Level 4 (https://www.w3.org/TR/css-color-4/#currentcolor-color), keyword is only mentioned in lowercase.
In CSS Color Module Level 3 (https://www.w3.org/TR/css-color-3/#currentcolor) keyword is written in camel case, and references SVG 1.0 specifications which also use camel case.
Finaly, SVG2 draft also mentions the keyword in camel case: https://svgwg.org/svg2-draft/painting.html#ColorProperty.

To summarize:

• CSS3, SVG1 and SVG2: currentColor
• CSS4: currentcolor

from stylelint-config-standard.

@rohm1 Thanks for the report. But I believe this is not a bug. You can avoid the error by the camelCaseSvgKeywords option:

{
  "rules": {
    "value-keyword-case": ["lower", {"camelCaseSvgKeywords": true}]
  }
}

See also this demo.

from stylelint-config-standard.

@ybiquitous thanks, but this is at least a default config bug. I should not have to configure anything for standard CSS properties and values.

from stylelint-config-standard.

Revisit old issues:

• stylelint/stylelint#1089
• stylelint/stylelint#4884
• stylelint/stylelint#5863

from stylelint-config-standard.

Just to confirm, are both valid, e.g., optimizeLegibility and optimizelegibility?

from stylelint-config-standard.

Not sure, the specs make only mention of the camel case option (https://svgwg.org/svg2-draft/painting.html#TextRenderingProperty). But both Firefox and Chrome seem to accept camel case and lowercase.

from stylelint-config-standard.

Umm, if we enable camelCaseSvgKeywords, the following valid CSS using the currentcolor keyword will be reported:

a {
  color: currentcolor;
/*       ^ Expected "currentcolor" to be "currentColor"
 */
}

See this demo.

What do you think?

from stylelint-config-standard.

Looks like this is wrong: https://github.com/stylelint/stylelint/blob/main/lib/reference/keywords.js#L265 currentcolor is expected to be camel case but the spec you mention states lowercase.

from stylelint-config-standard.

Does the SVG spec support currentColor, right?

from stylelint-config-standard.

Umm..., it seems we cannot enable camelCaseSvgKeywords until fixing the problem. 🤔

from stylelint-config-standard.

The CSS color 4 tests are all using currentColor:

https://github.com/web-platform-tests/wpt/blob/master/css/css-color/currentcolor-001.html
https://github.com/web-platform-tests/wpt/blob/master/css/css-color/currentcolor-002.html
https://github.com/web-platform-tests/wpt/blob/master/css/css-color/currentcolor-003.html

from stylelint-config-standard.

I came across this issue, and trying to trace down any upstream issue I found w3c/csswg-drafts#4599
which points over that the values are case insensitive https://drafts.csswg.org/css-values-4/#keywords, so fixing them to the camelCase should be safe

from stylelint-config-standard.