Skip to content

Interface: Increase footer breadcrumb height to prevent focus ring clipping - #81145

Merged
t-hamano merged 3 commits into
trunkfrom
update/interface-footer-height
Aug 4, 2026
Merged

Interface: Increase footer breadcrumb height to prevent focus ring clipping#81145
t-hamano merged 3 commits into
trunkfrom
update/interface-footer-height

Conversation

@t-hamano

@t-hamano t-hamano commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

What?

Prevents the focus ring from being clipped in the footer block breadcrumb.

Before

image

After

image

Why?

The focus ring changed from a box-shadow-based style to an outline-based one. Because outline-offset renders 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

  1. Open the editor and create nested blocks.
  2. Focus an item in the footer breadcrumb.
  3. Confirm the focus ring is fully visible and not clipped.

Use of AI Tools

Claude Code was used to apply the style changes and draft this description. All changes were reviewed by me.

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>
@github-actions github-actions Bot added [Package] Interface /packages/interface [Package] Base styles /packages/base-styles labels Aug 4, 2026
@t-hamano
t-hamano force-pushed the update/interface-footer-height branch from 647d640 to 3367b80 Compare August 4, 2026 10:18
@t-hamano t-hamano added [Type] Bug An existing feature does not function as intended Needs Design Feedback Needs general design feedback. labels Aug 4, 2026
@t-hamano
t-hamano requested review from a team August 4, 2026 10:23
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

Size Change: +334 B (0%)

Total Size: 7.81 MB

📦 View Changed
Filename Size Change
build/modules/boot/index.min.js 49.4 kB +15 B (+0.03%)
build/styles/edit-post/style-rtl.css 3.32 kB +8 B (+0.24%)
build/styles/edit-post/style-rtl.min.css 1.97 kB +15 B (+0.77%)
build/styles/edit-post/style.css 3.32 kB +7 B (+0.21%)
build/styles/edit-post/style.min.css 1.97 kB +16 B (+0.82%)
build/styles/edit-site/style-rtl.css 21.4 kB +7 B (+0.03%)
build/styles/edit-site/style-rtl.min.css 17.5 kB +7 B (+0.04%)
build/styles/edit-site/style.css 21.4 kB +7 B (+0.03%)
build/styles/edit-site/style.min.css 17.5 kB +7 B (+0.04%)
build/styles/edit-widgets/style-rtl.css 5.04 kB +24 B (+0.48%)
build/styles/edit-widgets/style-rtl.min.css 3.61 kB +29 B (+0.81%)
build/styles/edit-widgets/style.css 5.03 kB +23 B (+0.46%)
build/styles/edit-widgets/style.min.css 3.61 kB +31 B (+0.87%)
build/styles/editor/style-rtl.css 31.4 kB +14 B (+0.04%)
build/styles/editor/style-rtl.min.css 26.7 kB +22 B (+0.08%)
build/styles/editor/style.css 31.5 kB +15 B (+0.05%)
build/styles/editor/style.min.css 26.7 kB +22 B (+0.08%)
build/styles/media-utils/style-rtl.css 2.29 kB +15 B (+0.66%)
build/styles/media-utils/style-rtl.min.css 1.27 kB +17 B (+1.36%)
build/styles/media-utils/style.css 2.29 kB +16 B (+0.71%)
build/styles/media-utils/style.min.css 1.27 kB +17 B (+1.36%)

compressed-size-action

@t-hamano t-hamano self-assigned this Aug 4, 2026
@t-hamano t-hamano added the Backport to WP 7.1 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Aug 4, 2026
@t-hamano
t-hamano marked this pull request as ready for review August 4, 2026 10:26
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

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 props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

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>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@jasmussen

Copy link
Copy Markdown
Contributor

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 small?

@jasmussen

Copy link
Copy Markdown
Contributor

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?

@t-hamano

t-hamano commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the PR!

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.

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 mirka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread packages/base-styles/_mixins.scss Outdated
@mixin snackbar-container() {
position: fixed;
bottom: 24px;
bottom: 32px;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These might be better kept in sync if we used --wpds-dimension-size-md?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in 771e23b

display: flex;
background: $white;
height: $button-size-small;
height: $button-size-compact;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed in ac73dd2

t-hamano and others added 2 commits August 4, 2026 21:54
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>
@t-hamano
t-hamano merged commit 9043ec1 into trunk Aug 4, 2026
61 of 63 checks passed
@t-hamano
t-hamano deleted the update/interface-footer-height branch August 4, 2026 13:36
@github-project-automation github-project-automation Bot moved this from 🔎 Needs Review to ✅ Done in WordPress 7.1 Editor Tasks Aug 4, 2026
@github-actions github-actions Bot added this to the Gutenberg 23.8 milestone Aug 4, 2026
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

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.

# Checkout the wp/7.1 branch instead of trunk.
git checkout wp/7.1

# Create a new branch for your PR.
git checkout -b my-branch

# Cherry-pick the commit.
git cherry-pick 9043ec14b2064c6db5b604e4fdddc71bbf854e30

# Check which files have conflicts.
git status

# Resolve the conflict...
# Add the resolved files to the staging area.
git status
git add .
git cherry-pick --continue

# Push the branch to the repository
git push origin my-branch

# Create a PR and set the base to the wp/7.1 branch.
# See https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-base-branch-of-a-pull-request.

t-hamano added a commit that referenced this pull request Aug 4, 2026
…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>
@t-hamano

t-hamano commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Backported: #81156

@t-hamano t-hamano removed the Backport to WP 7.1 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Aug 4, 2026
@t-hamano t-hamano added the Backported to WP Core Pull request that has been successfully merged into WP Core label Aug 4, 2026
shail-mehta pushed a commit that referenced this pull request Aug 12, 2026
…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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backported to WP Core Pull request that has been successfully merged into WP Core Needs Design Feedback Needs general design feedback. [Package] Base styles /packages/base-styles [Package] Interface /packages/interface [Type] Bug An existing feature does not function as intended

Projects

Development

Successfully merging this pull request may close these issues.

3 participants

Sponsor
SponsoredKunjungi sekarang
Promo