#62313 closed defect (bug) (fixed)
Slight side-scroll in sidebar in WordPress 6.7 RC1
Reported by: | laurelfulford | Owned by: | joedolson |
---|---|---|---|
Milestone: | 6.7.1 | Priority: | normal |
Severity: | normal | Version: | 6.7 |
Component: | Customize | Keywords: | has-patch fixed-major commit dev-reviewed |
Focuses: | css | Cc: |
Description
In WordPress 6.7 RC1, there's a slight side-scroll in the Customizer sidebar.
I can recreate this with Twenty Nineteen and no other plugins activated.
I can't recreate this when the site is running WordPress 6.6.2.
Steps to reproduce
- Start on a site running the latest WP 6.7 RC.
- Install a theme that will load the Customizer (like Twenty Nineteen).
- Open the Customizer and note the appearance of the panel. If you don't have scrollbars set to be visible on Mac, you may need to try to scroll sideways inside of the sidebar to reproduce.
I was able to reproduce on OS 15.0, in Firefox (v. 131.0.3), Safari (v. 18.0), and Chrome (v. 129.0.6668.101).
Attachments (4)
Change History (27)
#2
@
5 weeks ago
patch.diff solves the problem by simply adding box-sizing:border-box
only. Alternatively, removing all:unset
and adding some missing styles would also solve the problem.
This ticket was mentioned in Slack in #core by wildworks. View the logs.
4 weeks ago
#6
@
4 weeks ago
Hi, there is an additional UX issue with regards to this whereby when the button text is long enough it is shown behind the arrow at the end so actually, the padding value also needs to be updated from padding: 10px 10px 11px 14px;
to padding: 10px 26px 11px 14px;
.
Oliver
#7
@
4 weeks ago
@domainsupport Thanks for the additional information. It seems the padding issue isn't new to WP6.7. I tested it with Playground and it's been there since at least WordPress 6.3.
@
4 weeks ago
Launch WP6.3 in the playground. If I access the customizer and apply long text via the browser console, the text overlaps the chevron icon.
#8
@
4 weeks ago
Yes, that's precisely what I'm talking about :)
I'll upload an updated patch accordingly.
#9
@
4 weeks ago
I think the scrollbar and padding issues need to be addressed separately. The scrollbar issue first appeared in WP 6.7, so I think it needs to be fixed in 6.7 major or minor release.
On the other hand, the padding issue is not new to 6.7, so it might be better addressed separately in #62335.
#10
@
4 weeks ago
OK, I'll add a patch to the other ticket. Although, this one will need to be patched first in order for #62335 to fix the overlap issue.
#11
@
3 weeks ago
- Focuses css added
- Keywords has-patch added
- Milestone changed from Awaiting Review to 6.7.1
In addition to the horizontal scrolling with the customize-pane-parent
panel, the chevron icon itself is not clickable for the Customizer buttons. If the box-sizing
changes for all of these Customizer buttons, then the Menus and Widgets panel buttons would have even less clickable area because the heading still has 30px
padding (on the right).
I was able to make the icon pseudo-element clickable with pointer-events: none
, but I am open to other ideas.
I think the Customizer button padding should be fixed on this ticket so both parent and child panels would avoid overlap in the same way, but a separate ticket could adjust padding for the Menus screen's buttons (which is not a new issue).
This ticket was mentioned in PR #7746 on WordPress/wordpress-develop by @sabernhardt.
3 weeks ago
#12
- Adds
box-sizing: border-box
to the buttons so they do not cause a horizontal scroll. - Increases specificity to remove padding from headings in the Menus and Widgets panels.
- Makes the chevron icon clickable.
- Adds padding to the buttons to avoid overlapping the chevron (30 pixels, to match what the child panel headings had).
- Adds a semicolon in
common.css
.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
2 weeks ago
#14
@
2 weeks ago
- Keywords commit added
The attached PR is looking good. @joedolson are you able to do a final review and commit?
#18
@
2 weeks ago
- Keywords dev-feedback added; commit removed
- Resolution fixed deleted
- Status changed from closed to reopened
Re-opening for dev sign off for 6.7.1.
Thanks for the report.
This issue is due to #42002. Also, this issue first appeared in WP 6.7, so I expect it will need to be fixed in the WP 6.7 release.