[wp-trac] [WordPress Trac] #26905: Follow CSS coding standards for SASS/SCSS files

[WordPress Trac]

#26905: Follow CSS coding standards for SASS/SCSS files
-----------------------------------+------------------------------
 Reporter:  morganestes            |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  Awaiting Review
Component:  General                |     Version:  trunk
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |
-----------------------------------+------------------------------
Changes (by morganestes):

 * keywords:   => has-patch 2nd-opinion


Comment:

 I'm new to SASS/SCSS, but having used LESS in several other projects I
 understand the concepts. I'm a bit torn on what to think about spacing in
 function calls. I'm used to doing it in PHP so it looks normal to see
 something like `lighten( $link, 10% )`, but when it gets to `desaturate(
 lighten( $menu-background, 7% ), 7% )` it starts looking a bit weird to me
 next to other CSS rules.

 The current CSS Coding Standards say **no** to padded spaces within things
 like `rgb()` and `url()`; I've never thought of them as function calls
 before, but it makes sense to follow that style for now with SASS
 functions that have the same syntax.

 I think we may need to have a new standard for the language, or include
 SCSS syntax into the current CSS coding standard so that all new files
 follow a pattern that new contributors can easily read and understand. A
 lot of devs are going to learn how to write code from reading what's in
 core, so this is a chance to influence them for good.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/26905#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform