Opened 10 months ago
Closed 10 months ago
#60707 closed defect (bug) (fixed)
Dropdown menus in Admin bar need more padding in WordPress 6.5 RC1
Reported by: | bgnicolepaschen | Owned by: | audrasjb |
---|---|---|---|
Milestone: | 6.5 | Priority: | normal |
Severity: | normal | Version: | 6.5 |
Component: | Toolbar | Keywords: | has-patch has-testing-info has-screenshots commit dev-reviewed fixed-major |
Focuses: | ui, accessibility, administration | Cc: |
Description (last modified by )
When using a non-block theme, the dropdown menus that appear in the admin bar on the front end of the site need more padding. When hovering on +New and the Site Title these dropdown menus appear.
Environment
- WordPress: 6.5-RC1-57778
- PHP: 7.4.33
- Server: Apache
- Database: mysqli (Server: 10.11.7-MariaDB-log / Client: mysqlnd 7.4.33)
- Browser: Chrome 122.0.0.0 (Windows 10/11)
- Theme: Twenty Twenty 2.5
- MU-Plugins:
- BoldGrid Demo 1.5.3
- BoldGrid Demo Housekeeping 1.0.0
- BoldGrid LogRocket 1.0.3
- Plugins:
- WordPress Beta Tester 3.5.5
Steps to Reproduce
- Use Twenty Twenty theme
- Navigate to the front end of the site.
- Hover on the Site Title or + New
- The links that appear do not have enough padding between them.
- 🐞 Bug occurs.
Expected Results
- I should see the standard amount of padding that stable has.
Actual Results
- ❌ The links are very close together
Attachments (5)
Change History (15)
#1
@
10 months ago
- Component changed from General to Toolbar
- Description modified (diff)
- Keywords has-patch added
- Milestone changed from Awaiting Review to 6.5
- Version set to trunk
Thanks for the report!
It's not padding; it's the height. [57765] changed the height to the rem equivalent, which does not work on the front end with themes that reset the rem unit.
Twenty Fifteen and Twenty Twenty use html { font-size: 62.5%; }
, and many of the themes in a directory search for a 62.5% font-size do the same. Twenty Nineteen is the opposite, with a larger base size of
html { font-size: 22px; }
.
#2
@
10 months ago
Reproduction Report
Description
I was able to reproduce this issue.
Environment
- WordPress: 6.5-RC1
- PHP: 8.3.2
- Server: nginx/1.25.3
- Database: mysqli (Server: 11.2.2-MariaDB / Client: mysqlnd 8.3.2)
- Browser: Chrome 122.0.0.0
- OS: macOS
- Theme: Twenty Twenty 2.5
- MU Plugins: None activated
- Plugins:
- Test Reports 1.1.0
#4
@
10 months ago
- Keywords has-testing-info has-screenshots added
Thanks for the patch, @sabernhardt!
Test Report
Patch tested: https://core.trac.wordpress.org/attachment/ticket/60707/60707.diff
Environment
- Hardware: MacBook Pro Apple M1 Pro
- OS: macOS 14.3.1
- Browser: Safari 17.3.1, Google Chrome 122.0.6261.111
- Server: nginx/1.25.4
- PHP: 8.2.16
- MySQL: 8.0.27
- WordPress: 6.5-RC1-57778-src
- Theme: twentytwenty v2.5
Actual Results
- ✅ Spacing between menubar subitems is restored (same as when block theme is active).
Supplemental Artifacts
Figure 1: Before patch ❌.
Figure 2: After patch ✅.
#5
@
10 months ago
Test Report
Patch Tested: https://core.trac.wordpress.org/attachment/ticket/60707/60707.diff
Environment:
WordPress - 6.5-RC1
OS - Windows
Browser - Firefox
Theme: Twenty Twenty
PHP - 8.1.23
Active Plugin - Test Reports
Actual Results:
- Spacing issue between menubar subitems is Resolved With Patch .✅
#7
@
10 months ago
- Keywords fixed-major dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
Reopening for backport. A second committer review is needed before backporting this to the 6.5 branch.
#8
@
10 months ago
- Keywords commit dev-reviewed added; fixed-major dev-feedback removed
@audrasjb LGTM for 6.5
restores pixel dimensions