Opened 5 weeks ago
Closed 3 weeks ago
#65062 closed defect (bug) (fixed)
Revisions: buttons are misplaced
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 7.0 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Revisions | Keywords: | admin-reskin has-patch has-test-info dev-reviewed |
| Focuses: | Cc: |
Description
Related to #64547
We changed the default size of most buttons from 30px to 40px. As a result, the space below the "Previous", "Next", and "Restore This Revision" buttons disappeared on the classic revision screen.
Attachments (1)
Change History (16)
This ticket was mentioned in PR #11549 on WordPress/wordpress-develop by @wildworks.
5 weeks ago
#1
- Keywords has-patch added
This ticket was mentioned in Slack in #core-test by gaisma22. View the logs.
5 weeks ago
#6
@
4 weeks ago
- Keywords has-test-info added
Patch Testing Report
Patch Tested: https://github.com/WordPress/wordpress-develop/pull/11549
## Environment
- WordPress: 7.1-alpha (latest)
- PHP: 8.2
- Browser: Chrome
- OS: macOS
- Theme: Twenty Twenty-Five
- Plugins: None activated
## Testing Steps
- Opened a post/page in WordPress admin
- Navigated to the Revisions screen
- Checked the layout in desktop, tablet, and mobile screen sizes
- Applied the patch and repeated the same steps
## Results
- The Previous and Next buttons alignment looks improved
- Layout appears consistent across different screen sizes
- Restore button is displayed correctly
- No functional issues observed
The patch is working as expected.
On smaller mobile screens, the buttons still appear slightly small, which could be improved further in future updates, but overall the current patch resolves the main alignment issue.
Tested across multiple screen sizes and everything is working fine.
This ticket was mentioned in Slack in #core-test by moksha_shah. View the logs.
4 weeks ago
@shailu25 commented on PR #11549:
4 weeks ago
#8
@peterwilsoncc commented on PR #11549:
3 weeks ago
#9
@wildworks commented on PR #11549:
3 weeks ago
#10
@peterwilsoncc
Viewing the button on a desktop, the margin above the button is noticeably larger than the margin below the button.
It might be due to CSS caching. Can you try a hard refresh of your browser?
Oddly, on a mobile view the reverse appears to be true.
The bottom margin in the mobile view was not addressed in this PR as it was already present. Screenshot from 6.9:
@peterwilsoncc commented on PR #11549:
3 weeks ago
#11
It might be due to CSS caching. Can you try a hard refresh of your browser?
Sorry Aki, it was that or something else PEBKAC.
This is good for commit.
#13
@
3 weeks ago
- Keywords dev-feedback added
- Resolution fixed deleted
- Status changed from closed to reopened
Trac ticket: #65062
## Use of AI Tools
None
## Screenshots
| 360px |
|
|
###