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 |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Summary
Merge Risk: ⚪ Minimal · up to
|
b243d39 to
efae034
Compare
) * Block Directory: Standardize focus rings on downloadable blocks * Block Directory: Document the downloadable block focus update * Block Directory: Suppress the hover box-shadow when a focused item is also hovered * Block Directory: Add PR reference to changelog * Block Directory: Use subtle background tint on hover and add border-radius --- Co-authored-by: himanshupathak95 <abcd95@git.wordpress.org> Co-authored-by: ciampo <mciampini@git.wordpress.org>
|
I just cherry-picked this PR to the wp/7.2 branch to get it included in the next release: 0741b7f |
|
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. |
|
@himanshupathak95 , can you please open a new PR to manually backport this change to the 7.2 branch? (cc @t-hamano ) |
Yes, I can. Coming right up. |
|
Actually, the backport succeeded. Adding two labels at once probably fires two events, so the workflow ran twice. I haven't looked into it in detail, but we probably need to adjust the workflow. |
|
Oh, I missed the earlier message 🤦 Sorry for the ping, you both 🙏 |
What?
Part of #80138. Use the design system focus outline for downloadable-block cards in the inserter.
Why?
These cards still use the old shadow for keyboard focus. Their focus ring can also be clipped when arrow-key navigation scrolls a card against the bottom edge.
How?
Use
outset-ring__focus()for keyboard focus and a subtle neutral background tint on hover. Remove the old hover shadow and its transition, use the shared small corner radius, and keep the scroll margin that prevents the focus outline from being clipped. Disabled cards do not get the hover tint.Testing Instructions
map. Wait for “Available to install”. This needs access to WordPress.org and a block that is not already installed.Testing Instructions for Keyboard
map.Screenshots or screencast
Before
After
Screen.Recording.2026-10-22.at.12.13.26.mov
Use of AI Tools
Implemented with assistance of Codex (Astra 6)
@coderabbitai review