Problem/Motivation

Follow-up to #2865971: Use stylelint as opposed to csslint in core. Configure the color-hex-case to be consistent with https://www.drupal.org/docs/develop/standards/css/css-coding-standards

From the CSS Formatting Guidelines:

When hex values are used for colors, use lowercase and, if possible, the shorthand syntax, e.g. #aaa. Colors may be expressed with any valid CSS value, such as hex value, color keyword, rgb() or rgba(). Note that IE8 does not support all color syntaxes and will require a fallback value.

Proposed resolution

Brief instructions on running stylelint - you'll need npm...

All the commands below take place in DRUPAL_ROOT/core
To install stylelint

npm install

This will install Drupal 8's npm dependencies of which stylelint is one.

To run it on all core css files. Apply this issue's patch and do the following command from DRUPAL_ROOT/core

npm run lint:css

Remaining tasks

User interface changes

None

API changes

None

Files: 

Comments

alexpott created an issue. See original summary.

alexpott’s picture

Issue summary: View changes
BrightBold’s picture

Issue summary: View changes
droplet’s picture

Version: 8.3.x-dev » 8.4.x-dev
Status: Active » Needs review
Issue tags: +stylelint
FileSize
25.86 KB

Status: Needs review » Needs work

The last submitted patch, 5: 2866801-4.patch, failed testing.