search

xojs / stylelint-config-xo

Stylelint shareable config for XO
MIT License
42 stars 6 forks source link

Allow -moz-tab-size #10

Closed fregante closed 4 years ago

fregante commented 4 years ago

I'm not sure of what's the logic for allowing properties here, but how about -moz-tab-size?

Prefix-less tab-size is still not allowed in Firefox.

sindresorhus commented 4 years ago

It's already defined here: https://github.com/xojs/stylelint-config-xo/blob/46439f985f1b83f65d6e8925cf4278e551cb8a48/index.js#L85

fregante commented 4 years ago

value-no-vendor-prefix is not what's triggering this error:

https://github.com/sindresorhus/refined-github/runs/658808320#step:4:10

source/refined-github.css
 16:2  ✖  Unexpected vendor-prefix "-moz-tab-size"   property-no-vendor-prefix

I think value-no-vendor-prefix allows display: -moz-flex while property-no-vendor-prefix allows -moz-tab-size: 2

sindresorhus commented 4 years ago

You're right. Can you remove that previous tab-size entry then? Since it's in the wrong place.