Stop enforcing a11y/content-property-no-static-value, which feels too restrictive for projects already following best practices

Author: thibaudcolas

CHANGELOG.md

@@ -6,6 +6,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 
 ## Unreleased
 
+### Features
+
+- Stop enforcing `a11y/content-property-no-static-value`, which feels too restrictive for projects already following best practices.
+
 ### Bug fixes
 
 - Fix `scale-unlimited/declaration-strict-value` being applied twice to `background-color` declarations.

README.md

@@ -73,7 +73,6 @@ To get the most out of this config, it is assumed that projects have the followi
 - [`scss/at-mixin-argumentless-call-parentheses`](https://github.com/kristerkari/stylelint-scss#readme): `always`
 - [`scss/at-mixin-pattern`](https://github.com/kristerkari/stylelint-scss#readme): ``
 - [`order/order`](https://github.com/hudochenkov/stylelint-order): `dollar-variables, custom-properties, type: at-rule, hasBlock: false, declarations`
-- [`a11y/content-property-no-static-value`](https://github.com/YozhikM/stylelint-a11y#readme)
 - [`a11y/no-obsolete-attribute`](https://github.com/YozhikM/stylelint-a11y#readme)
 - [`a11y/no-obsolete-element`](https://github.com/YozhikM/stylelint-a11y#readme)
 - [`a11y/no-text-align-justify`](https://github.com/YozhikM/stylelint-a11y#readme)

config.js

@@ -54,7 +54,6 @@ module.exports = {
             { type: 'at-rule', hasBlock: false },
             'declarations',
         ],
-        'a11y/content-property-no-static-value': true,
         'a11y/no-obsolete-attribute': true,
         'a11y/no-obsolete-element': true,
         'a11y/no-text-align-justify': true,

src/__snapshots__/semver.test.js.snap

@@ -2,9 +2,6 @@
 
 exports[`semver - should those tests break, consider releasing a new major version of the package config contents 1`] = `
 Object {
-  "a11y/content-property-no-static-value": Array [
-    true,
-  ],
   "a11y/no-obsolete-attribute": Array [
     true,
   ],

src/unused.js

@@ -61,6 +61,9 @@ const tooOpinionated = [
     'scss/dollar-variable-default',
     'scss/no-dollar-variables',
     'order/properties-alphabetical-order',
+    // This is useful in principle, but a blanket ban means lots of manual work to add "disable"
+    // comments whenever `content` is used for decorative purposes only, e.g. breadcrumb / menu separators.
+    'a11y/content-property-no-static-value',
     'a11y/font-size-is-readable',
     'a11y/line-height-is-vertical-rhythmed',
     'a11y/media-prefers-color-scheme',