[buddypress-trac] [BuddyPress Trac] #7028: Use stylelint to lint SCSS & CSS replacing Ruby gem `scss_lint`

[buddypress-trac]

#7028: Use stylelint to lint SCSS & CSS replacing Ruby gem `scss_lint`
---------------------------------------+-----------------------------
 Reporter:  netweb                     |       Owner:
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  Future Release
Component:  Tools - Build Process      |     Version:  2.3.0
 Severity:  normal                     |  Resolution:
 Keywords:  needs-patch needs-refresh  |
---------------------------------------+-----------------------------

Comment (by hnla):

 >Yes, the coding standards state: "Use shorthand (except when overriding
 styles) for background, border, font, list-style, margin, and padding
 values as much as possible. (For a shorthand reference, see CSS
 Shorthand."

 And how does this automated test determine when styles are attempting to
 override something that perhaps is located somewhere in the Cascade?

 The reference to 'shorthand' actually covers both:

 ``` property: 0 0 0;```

 and

 ``` property: 0 0 0 0;```

 Programatically it is next to impossible to tell why one is used over
 another and neither is actually incorrect ( this is very similar to an
 issue that always used to exist in the jigsaw validator).

 Aspects like this of CSS coding are tricky and where we always preferred
 that people would write `0 0 0` especially if obvious this was perhaps the
 only occurrence of this property on a selector, in larger scenarios it
 becomes harder to enforce this.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7028#comment:7>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac