Make WordPress Core

#63831 closed defect (bug) (fixed)

Adjust sizing of page header links on mobile viewports

Reported by: joedolson Owned by: joedolson
Priority: normal Milestone: 6.9
Component: Administration Version: 6.4
Severity: normal Keywords: has-patch commit
Cc: Focuses: css

Description

In #41986, the styling of the page title actions was adjusted to match the global button style. However, this styling was not also applied to the mobile viewport, and the mobile version does not match; it has excessive height, in particular.

This will require change the padding to 0 14px, adding min-height of 40px, lilne-height of 2.71428571, and vertical-align middle in the mobile viewport.

Attributing this to 6.4, as that's the point where it should have been adjusted, though the styles themselves are considerably older.

Attachments (2)

Screenshot 2025-08-15 at 3.30.17 PM.png (56.1 KB ) - added by joedolson 12 months ago.
Before
Screenshot 2025-08-15 at 3.31.03 PM.png (56.4 KB ) - added by joedolson 12 months ago.
After

Download all attachments as: .zip

Change History (13)

#1 @joedolson
12 months ago

  • Owner set to joedolson
  • Status newaccepted

This ticket was mentioned in PR #9495 on WordPress/wordpress-develop by @joedolson.


12 months ago
#2

  • Keywords has-patch added; needs-patch removed

#3 @joedolson
12 months ago

I reversed the labels on those images. The first is after, the second is before.

@shailu25 commented on PR #9495:


12 months ago
#4

BeforeAfter
https://github.com/user-attachments/assets/95017aab-3b31-419c-90ab-049a5be6a7b0 https://github.com/user-attachments/assets/e04c6fe5-debd-49ed-8840-e3376c48723b

#5 @rollybueno
12 months ago

  • Keywords needs-testing added

#6 @rollybueno
12 months ago

Test Report

Description

This report validates whether the indicated patch works as expected.

Patch tested: https://github.com/WordPress/wordpress-develop/pull/9495

Environment

  • WordPress: 6.9-alpha-60093-src
  • PHP: 8.2.29
  • Server: nginx/1.29.0
  • Database: mysqli (Server: 8.4.6 / Client: mysqlnd 8.2.29)
  • Browser: Chrome 139.0.0.0 (tested using Chrome emulator)
  • OS: Linux
  • Theme: Twenty Twenty-Five 1.3
  • MU Plugins: None activated
  • Plugins:
    • Test Reports 1.2.0

Actual Results

  1. ✅ Issue resolved with patch.
  2. No errors or regressions encountered during testing.

Additional Notes

  • Patch applies cleanly against trunk.
  • No console or PHP warnings observed.
  • ⚠️ Tested using Chrome emulator. Results should be validated on real devices for accuracy.

Supplemental Artifacts

Before:
https://i.imgur.com/pJEVGvM.png

After:
https://i.imgur.com/J4npqGc.png

@sumitbagthariya16 commented on PR #9495:


12 months ago
#7

Test Report - ✅
---

Patch tested: https://github.com/WordPress/wordpress-develop/pull/9495
Test this pull request with WordPress Playground.

Environment
--

  • WordPress: WordPress 6.9-alpha-20250815.203032
  • PHP: 7.4.31-dev (Supports 64bit values)
  • Database: mysqli (Server: 8.4.6 / Client: mysqlnd 8.2.29)
  • Browser: Chrome 139.0.7258.139 (Official Build) (arm64) (tested using Chrome emulator)
  • OS: Linux
  • Theme: Twenty Twenty-Five 1.3

https://github.com/user-attachments/assets/2c3ea643-f325-41a7-a516-8162e2ddf23d

https://github.com/user-attachments/assets/951f9cfc-ce59-4b36-8f2b-c89987e398a7

@circlecube commented on PR #9495:


12 months ago
#8

Test Report

This report validates that the indicated patch addresses the issue.

Patch tested: pull/9495

Environment

  • OS: macOS 15.6
  • Web Server: Playground
  • PHP: 7.4.29
  • WordPress: 6.8.2
  • Browser: Chrome (latest)
  • Theme: Twenty twenty-five
  • Active Plugins: none

Actual Results

  • ✅ Issue resolved with patch.

Additional Notes

  • This looks good.

Supplemental Artifacts

Before (pre-patch) tested via local on 6.8.2 and PHP 7.4.
https://github.com/user-attachments/assets/c0c66ad8-c9bc-41cc-8b4b-4baadd0f41c0

Playground link (with patch)
https://github.com/user-attachments/assets/c112789a-3a1d-4561-b43e-5fc3029a7398

#9 @SirLouen
12 months ago

  • Keywords needs-testing removed

#10 @joedolson
11 months ago

  • Keywords commit added

#11 @joedolson
11 months ago

  • Resolutionfixed
  • Status acceptedclosed

In 60705:

Administration: Fix sizing of page header links on mobile.

Page title actions were modified to match the global button styles in #41986, but these styles were not applied to mobile viewports at the time.

Fix padding, height, line-height, and alignment to match these links to global styles.

Props joedolson, rollybueno, sumitbagthariya16, circlecube, shailu25.
Fixes #63831.

Note: See TracTickets for help on using tickets.