Consider replacing deprecated `clip`
Nobody has claimed this yet.
- Dominant language
- SCSS
- Stars
- 298
- Forks
- 83
- Avg merge
- 16h 48m
- Merged PRs (30d)
- 1
Description
Description
The visually-hidden helper includes an outdated syntax of clip: rect() that may need replacing with some other clipping properties or maybe updating the helper to something more current overall.
Steps to reproduce
The next stylelint-config-standard-scss major version won't like it.
😭 /actions/runs/17982808350/job/51153182926 ✂️
Expected result
https://developer.mozilla.org/en-US/docs/Web/CSS/clip
Actual result
Environment
This might be breaking of sorts, in case a consumer applies the mixin, but then e.g. redefines or undoes the visual hiding in some cases, changing the clip value there — so there might be a migration to the newer mixin content version necessary for them.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the visually-hidden helper in assets/sass/protocol/includes/mixins/_utils.scss at the referenced lines, then review the stylelint-config-standard-scss failure from the linked workflow. Check the linked Mozilla bedrock usages for assumptions about the current clip value. Done means the deprecated syntax is addressed, lint passes, and any consumer migration impact is documented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scss
- Domain
- frontend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100