Interface: Increase footer breadcrumb height to prevent focus ring clipping - #81145
Conversation
Change the interface skeleton footer height from `$button-size-small` (24px) to `$button-size-compact` (32px), and update the matching `.has-footer` padding offset. Also bump the snackbar container's bottom offset from 24px to 32px so snackbars keep clearing the footer. Co-Authored-By: Claude <noreply@anthropic.com>
647d640 to
3367b80
Compare
|
Size Change: +334 B (0%) Total Size: 7.81 MB 📦 View Changed
|
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If 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. |
|
The code looks good, well put together. However the footer is intentionally very small, so although I wouldn't want to block this from landing, I'm also not convinced the correct solution is to make the entire bar larger. Anecdotally I've heard feedback from users that they find the breadcrumbs bar annoying and confusing, they are usually happy when I tell them they can turn it off in Preferences. Can the buttons not be |
|
Correction, it might already be small, and stay small, and you may be increasing the size to avoid the crop of the focus style. This will not be the only context in which we see close-to-the-edge buttons (of any size) have their focus rings cut off. So I think the outline style change could need some context here. @mirka any thoughts? |
|
Thanks for the PR!
That's right. We have experience fixing similar issues in the past. In another PR, a negative outline-offset was exceptionally applied to prevent the focus ring from expanding outwards. |
mirka
left a comment
There was a problem hiding this comment.
This will not be the only context in which we see close-to-the-edge buttons (of any size) have their focus rings cut off. So I think the outline style change could need some context here. @mirka any thoughts?
Right. I think increasing the surrounding space like this is generally going to be the way to address this systematically. It's not like a focus ring that's perfectly flush with the outer container is going to look nice anyway. Interactive elements need a bit of breathing room.
| @mixin snackbar-container() { | ||
| position: fixed; | ||
| bottom: 24px; | ||
| bottom: 32px; |
There was a problem hiding this comment.
These might be better kept in sync if we used --wpds-dimension-size-md?
| display: flex; | ||
| background: $white; | ||
| height: $button-size-small; | ||
| height: $button-size-compact; |
There was a problem hiding this comment.
Since this is not really tied to a compact size button, we should probably use --wpds-dimension-size-md, or at least some kind of meaningful math like $button-size-small + --wpds-dimension-padding-sm.
Keeps the snackbar container's bottom offset in sync with the footer height by referencing the design token instead of a hardcoded 32px. Co-Authored-By: Claude <noreply@anthropic.com>
The footer is not tied to a compact-size button, so reference the design token directly instead of $button-size-compact. Co-Authored-By: Claude <noreply@anthropic.com>
|
There was a conflict while trying to cherry-pick the commit to the wp/7.1 branch. Please resolve the conflict manually and create a PR to the wp/7.1 branch. PRs to wp/7.1 are similar to PRs to trunk, but you should base your PR on the wp/7.1 branch instead of trunk. |
…ipping (#81145) * Interface: Increase footer height to 32px Change the interface skeleton footer height from `$button-size-small` (24px) to `$button-size-compact` (32px), and update the matching `.has-footer` padding offset. Also bump the snackbar container's bottom offset from 24px to 32px so snackbars keep clearing the footer. Co-Authored-By: Claude <noreply@anthropic.com> * Base Styles: Use --wpds-dimension-size-md for snackbar bottom offset Keeps the snackbar container's bottom offset in sync with the footer height by referencing the design token instead of a hardcoded 32px. Co-Authored-By: Claude <noreply@anthropic.com> * Interface: Use --wpds-dimension-size-md for the footer height The footer is not tied to a compact-size button, so reference the design token directly instead of $button-size-compact. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: mirka <0mirka00@git.wordpress.org> Co-authored-by: jasmussen <joen@git.wordpress.org>
|
Backported: #81156 |
…ipping (#81145) * Interface: Increase footer height to 32px Change the interface skeleton footer height from `$button-size-small` (24px) to `$button-size-compact` (32px), and update the matching `.has-footer` padding offset. Also bump the snackbar container's bottom offset from 24px to 32px so snackbars keep clearing the footer. Co-Authored-By: Claude <noreply@anthropic.com> * Base Styles: Use --wpds-dimension-size-md for snackbar bottom offset Keeps the snackbar container's bottom offset in sync with the footer height by referencing the design token instead of a hardcoded 32px. Co-Authored-By: Claude <noreply@anthropic.com> * Interface: Use --wpds-dimension-size-md for the footer height The footer is not tied to a compact-size button, so reference the design token directly instead of $button-size-compact. Co-Authored-By: Claude <noreply@anthropic.com> --------- Co-authored-by: t-hamano <wildworks@git.wordpress.org> Co-authored-by: mirka <0mirka00@git.wordpress.org> Co-authored-by: jasmussen <joen@git.wordpress.org>
What?
Prevents the focus ring from being clipped in the footer block breadcrumb.
Before
After
Why?
The focus ring changed from a
box-shadow-based style to anoutline-based one. Becauseoutline-offsetrenders the ring slightly outside the element, it no longer fits within the 24px footer height.How?
Increases the breadcrumb height from 24px to 32px.
I think this is acceptable, but if it isn't, we need to consider another approach — for example, a negative
outline-offset. That would, however, remove the rounded corners of the focus ring.Testing Instructions
Use of AI Tools
Claude Code was used to apply the style changes and draft this description. All changes were reviewed by me.