Nicole Sullivan
Nicole Sullivan
Hmm, this makes a good point. I think the rule may need to be more clever... e.g. check that each text and font property is declared only once per heading....
There are also good reasons for duplicating properties. For example, all the headings are the same weight except one: h1, h2, h3, h4, h5, h6 {font-face: Verdana;font-weight:bold;} h1 {font-size: 2.1em;}...
@hojberg - if we use normalize.css instead of reset.css, we won't have that problem. The headings in normalize are meant to be adapted for every project.
@mahonnaise - so you mean check for location dependent selectors like this? .foo h1{} .bar h1{}
Did you try building it with Grunt?
Overall, the pull request looks good. Can you please write up some documentation for the new rule? https://github.com/stubbornella/csslint/wiki/Rules
@arnogues - this PR won't automatically merge, can you update it?
I'd be tempted to ignore vendor prefixes in this rule, just because they aren't really duplication a developer can do anything about...
@arnogues - can you please put your doc code in this thread?
@mirisuzanne, that's right. @gregwhitworth and I are working together on this. Right now the goals are mainly a11y and a design refresh, but ultimately we'd both like to make things...