Coding styleguide and linting for Sass/CSS

eHealthAfrica / documentation

Documentation for eHealth Africa

0 stars 0 forks source link

Coding styleguide and linting for Sass/CSS #18

Closed admbtlr closed 8 years ago

admbtlr commented 8 years ago

Suggestion:

Use Sass in every project
Adoption of the Sass guidelines by Hugo Giraudel (http://sass-guidelin.es/#too-long-didnt-read)
Add grunt-scss-lint to Gruntfiles throughout and use the suggested ruleset
Defer discussion of naming conventions and structure (e.g. OOCSS/BEM/etc.) to separate frontend/design topic
Add best practices about inclusion of Bootstrap, the theme or similar vendor libraries

admbtlr commented 8 years ago

:+1:

admbtlr commented 8 years ago

:+1: :facepunch:

admbtlr commented 8 years ago

grunt-scss-lint has a Ruby dependency :-/

admbtlr commented 8 years ago

This might help: sass-lint (needs to be evaluated)

admbtlr commented 8 years ago

Evaluation: https://github.com/sasstools/sass-lint/blob/master/lib/sass-lint.js

admbtlr commented 8 years ago

haha omg.

admbtlr commented 8 years ago

@ally @adamvert @FrancescoK.

So we had some conversations around this; BEM seems to be the favoured model. Do you think we could move forward on this? Hugo's guidelines + BEM for new work?

admbtlr commented 8 years ago

Yes, there seems to be a consensus around BEM. Personally I have always been reluctant because BEM is kinda ugly and seems overkill considering the relatively low complexity of our designs, but that’s not really a rational argument. Imho, let’s go forward with BEM. How to call things within BEM will be the more interesting discussion but hey :)