Default Intracto ruleset

Intracto / stylelint-config-intracto

NPM package

0 stars 1 forks source link

Open timdujardin opened 4 years ago

timdujardin commented 4 years ago

Disable no-descending-specificity rule: I would request to disable this rule, since it reduces readability.

ioulian commented 4 years ago

disable color-named, always use colors as hex notation or rgba, hsl...

timdujardin commented 4 years ago

Enable font-weight-notation: 'numeric'? I would recommend using the numbers instead of the labels/names.

ioulian commented 4 years ago

comment-empty-line-before to "always" except: "first-nested"

ioulian commented 4 years ago

comment-whitespace-inside to always

ioulian commented 4 years ago

function-url-quotes to always

timdujardin commented 4 years ago

Enable no-unknown-animations? This to ensure that animations are setup correctly.

ioulian commented 4 years ago

timdujardin commented 4 years ago

Add unit-blacklist: ['px']? To prevent using px values instead of em or rem.

We could make some exceptions though, for borders of 1px for example.

timdujardin commented 4 years ago

Add selector-pseudo-element-colon-notation: 'double': To prevent using f.e. &:before instead of &::before.

timdujardin commented 4 years ago

Add shorthand-property-no-redundant-values: true: To prevent using margin: 1rem 1rem 1rem 1rem instead of margin: 1rem.

ioulian commented 4 years ago

Ok, added:

To discuss:

unit-blacklist: there are a lot of use cases to use px values, we can't just blacklist them all
font-weight-notation: is a good idea, the problem is that some people won't know how to use this (700 = bold, 600 = medium...)
no-descending-specificity: I would not disable this as this is how the parser interprets this, and you can see the specificity in your code

Still to implement: