trendwerk / sphynx

A light & simple WordPress starter theme 😽
16 stars 2 forks source link

Remove underscores from .sass files. #788

Closed sboerrigter closed 7 years ago

sboerrigter commented 7 years ago

I prefer to remove the leading underscores from Sass files and instruct Gulp to only compile main.scss and editor.scss. This makes the file structure cleaner and prevents errors with forgotten underscores.

haroldangenent commented 7 years ago

I feel that disregarding Sass' own guidelines wouldn't be a good choice for a framework / starter theme.

luukdv commented 7 years ago

Disagreed, for the same reason. The underscores are also handy for lookup, and have special meaning (they're not compiled to an output file for example).