Closed Bug 1988860 Opened 8 months ago Closed 6 months ago

Add a Stylelint rule to enforce using size tokens

Categories

(Developer Infrastructure :: Lint and Formatting, task)

Tracking

(firefox146 fixed)

RESOLVED FIXED
146 Branch
Tracking Status
firefox146 --- fixed

People

(Reporter: hjones, Assigned: oarnesto)

References

(Blocks 1 open bug)

Details

(Whiteboard: [recomp][acorntractors])

Attachments

(1 file)

We should add a Stylelint rule to enforce only using --size-* tokens for any height, width, min-height, min-width, and maybe even background-size (?) properties in CSS and SCSS files. This rule is likely autofixable. This work will include documentation, tests, and adding an entry to our rollouts file, but will not include expanding our token offerings or fixing any existing errors.

Blocks: 1988862
No longer blocks: 1988862
Blocks: 1988863
No longer blocks: 1988863
Blocks: 1988871
Assignee: nobody → oarnesto
Status: NEW → ASSIGNED
Blocks: 1995685
Pushed by oarnesto@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/2b75d601c974 https://hg.mozilla.org/integration/autoland/rev/e9a25ec36cac stylelint rule to enforce using size design tokens r=frontend-codestyle-reviewers,akulyk,Gijs
Pushed by abutkovits@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/9d7920d565f5 https://hg.mozilla.org/integration/autoland/rev/9a6257287823 Revert "Bug 1988860 - stylelint rule to enforce using size design tokens r=frontend-codestyle-reviewers,akulyk,Gijs" for causing stylelint failures.
Pushed by oarnesto@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/b7da164f64ec https://hg.mozilla.org/integration/autoland/rev/4933f1406d6a stylelint rule to enforce using size design tokens r=frontend-codestyle-reviewers,akulyk,Gijs
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 146 Branch
Flags: needinfo?(oarnesto)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: