Repository navigation
Conversation
🤖 PR meta 🤖🎉 PropsIf you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Updated as activity occurs, without notifying anyone named here. Add the |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
ciampo
left a comment
There was a problem hiding this comment.
Thank you!
While we're here, a few extra changes that we can make. We could also remove CheckboxControl's redundant initial border and border-radius declarations and first &:checked block, plus RadioControl's duplicate opacity: 1? Keep the :disabled:checked::before opacity override.
For the radio, could we add border-color: var(--wpds-color-stroke-interactive-neutral) to the focus block and move it before the hover, checked, and disabled rules? That prevents the existing forms.css focus rule from darkening an unchecked radio's border while preserving the other states.
b23a2f5 to
36b7ae4
Compare
|
There was a conflict while trying to cherry-pick the commit to the wp/7.2 branch. Please resolve the conflict manually and create a PR to the wp/7.2 branch. PRs to wp/7.2 are similar to PRs to trunk, but you should base your PR on the wp/7.2 branch instead of trunk. |
|
I think the automatic cherry-pick failed on this one. |
|
It did. would you be able to create a manual cherry pick yourself? Cc @t-hamano |
Yes, I can do that. |
|
In the first place, this PR is an enhancement, but should we backport it? |
…85436) * Components: Use shared checkbox and radio focus outlines * Components: Document checkbox and radio focus ring changes * Components: Link PR 85379 to checkbox and radio focus changelog * CheckboxControl: Remove redundant input styles * RadioControl: Preserve state borders on focus --- (cherry picked from commit 9345e88) Co-authored-by: himanshupathak95 <abcd95@git.wordpress.org> Co-authored-by: ciampo <mciampini@git.wordpress.org>
|
This PR was backported to wp/7.2 by #85436. |
What?
See #80138. Use the shared outline focus ring in
CheckboxControlandRadioControl.Why?
Both controls still draw focus rings with box shadows. The existing rings were intentional, as described in #50127. This keeps the gap around each control while moving its ring to the shared design system style.
How?
input-controlmixin supplies the outline.outset-ring__focus()and clear the wp-admin focus shadow.The shared ring uses the focus-width token, including its fallbacks. Checked, mixed, and disabled styling stays the same.
Testing Instructions
npm run --workspace @wordpress/storybook storybook:dev.Testing Instructions for Keyboard
Screenshots or screencast
Before
Screen.Recording.2026-10-22.at.15.25.29.mov
Screen.Recording.2026-10-22.at.15.28.57.mov
After
Screen.Recording.2026-10-22.at.15.07.32.mov
Screen.Recording.2026-10-22.at.15.11.14.mov
Use of AI Tools
Implemented with assistance of Codex (Astra 6)
@coderabbitai review