Opened 3 years ago
Last modified 4 months ago
#58335 new defect (bug)
WordPress Dashboard Collapse Menu Transition issue.
| Reported by: | shuvo52 | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | Awaiting Review |
| Component: | Administration | Version: | 5.2 |
| Severity: | normal | Keywords: | has-screenshots has-patch |
| Cc: | Focuses: | ui, css |
Description
Hi
When I collapse the menu it's showing awkward.
See the video: https://d.pr/v/34K9H4
I think this issue should be fixed as soon as possible.
Regards
Attachments (2)
Change History (8)
#1
@
3 years ago
- Component General → Administration
- Focuses accessibility removed
- Version trunk → 5.2
This ticket was mentioned in Slack in #core-test by ironprogrammer. View the logs.
3 years ago
#3
@
3 years ago
- Priority normal → low
During test triage, it was agreed to mark this ticket's priority to low, but keep it open for a patch. Why low? Triagers noted that the sub-pixel shift was barely noticeable. However, the assessment in comment:1 is a good basis for a patch.
#4
@
3 years ago
- Keywords has-screenshots added
Adding has-screenshots as reporter's video does show the shift.
#5
@
15 months ago
- Keywords has-patch added; needs-patch removed
Summary of patch added:
wp-admin/css/admin-menu.css
- Lines 237-238: Added
box-sizing: border-box;andheight: 34px;to the#adminmenu a.menu-top, #adminmenu .wp-submenu-headselector to fix the 0.2px issue - Lines 322-323: Updated
.folded #adminmenu div.wp-menu-imagefromwidth: 35px; height: 30px;towidth: 36px; height: 34px;to maintain consistency
This patch addresses the core issue in ticket #58335 by:
- Standardizing menu item heights to exactly 34px in both collapsed and expanded states
- Eliminating the visual "jump" during menu transitions
#6
@
4 months ago
Related: #65266
- PR 11860 edits the
line-heightto reduce the calculated height. - 58335.1.diff deletes the width and height rules for the
.foldedstyle. [26766] reduced the.foldedwidth to 35px for long link text in #26452, but the text wraps to another line more smoothly now.
However, I think the 34-pixel width is worth keeping, between the 782px and 960px CSS breakpoints. That seems to center the icon better horizontally within the narrow column and align it with the W logo in the toolbar.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Thanks for the report!
The icon dimensions have a discrepancy between the expanded and collapsed states. The expanded link's height increased by 0.2 pixels in [45186], and the width had a 1-pixel difference before then. This is more noticeable on the About pages than any page where you would find an open submenu on the side (because expanding a submenu is a bigger difference).
line-heightplus 8px padding on the top and bottom).