Problem/Motivation
Blocker to #2865971: Use stylelint as opposed to csslint in core
We've got lots of whitespace issues in CSS were we are not adhering to https://www.drupal.org/docs/develop/standards/css/css-coding-standards
Proposed resolution
Fix whitespace issues in CSS using #2865971: Use stylelint as opposed to csslint in core to detect it.
To review the patch use git diff --color-words --word-diff-regex=.
. It makes it easy to see the scope of the changes are only whitespace.
Remaining tasks
This is split off from #2865971: Use stylelint as opposed to csslint in core - the very minor change is that
/* border-top: 1px solid #bfbfbf; */
Is fixed to add spaces so that it passes the ruleset rather than removed.
User interface changes
None
API changes
None
Comment | File | Size | Author |
---|---|---|---|
#2 | 2878548-2.patch | 39.71 KB | alexpott |
Comments
Comment #2
alexpottIn discussion with @xjm as this is part of an rtbc patch it can go straight to rtbc.
Comment #9
alexpottAdding credit from #2865971: Use stylelint as opposed to csslint in core
Comment #10
Anonymous (not verified) CreditAttribution: Anonymous commentedComment #11
alexpottComment #13
cilefen CreditAttribution: cilefen commentedCommitted ef1db63 and pushed to 8.4.x. Thanks!
This completely satisfies stylelint with the patch from #2865971: Use stylelint as opposed to csslint in core, and it's easier to read. Go standards!