Make WordPress Core

Opened 9 months ago

Last modified 5 months ago

#58844 new defect (bug)

Revisions: Compare Revisions layout issues on mobile

Reported by: plari's profile plari Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.6
Component: Revisions Keywords: has-screenshots good-first-bug has-patch
Focuses: Cc:

Description

This is a follow-up to #33830.

"Restore This Revision" button in wp-admin revision page can become hidden behind the diff panel in mobile view, particularly if the post author has a longer name.

The problem occurs when the author info (Name & Timestamp) stacks below the Avatar.

Attachments (1)

Screenshot on 2023-07-18 at 13-44-51.png (47.4 KB) - added by plari 9 months ago.

Download all attachments as: .zip

Change History (10)

#1 @adamsilverstein
5 months ago

  • Milestone changed from Awaiting Review to Future Release

Hi @plari - thanks for opening the ticket. Good catch, looks like a style issue.

Do you know what device/width were you testing with?

#2 @adamsilverstein
5 months ago

  • Keywords needs-patch has-screenshots good-first-bug added

#3 @adamsilverstein
5 months ago

  • Version changed from 6.2.2 to 3.6

#4 @plari
5 months ago

Hey @adamsilverstein! That screenshot I shared was in Firefox using in-browser responsive tools; I think the width really depends on the length of the Public display name – as soon as the edge of the window reaches the name, the name stacks below the avatar, and the button moves down as well. Retested today, and that happened at 456px. I can produce the same result viewing on Safari, iPhone XR. Hope this helps!

#5 @adamsilverstein
5 months ago

Thanks for the details @plari. I don't think we have tested extensively on mobile so we probably just never noticed the issue, thanks for reporting.

I've marked this as a good-first-bug since I feel like the fix should be pretty straightforward.

This ticket was mentioned in PR #5714 on WordPress/wordpress-develop by ifaisaldev.


5 months ago
#6

  • Keywords has-patch added; needs-patch removed

…dden behind the diff panel in mobile view, particularly if the post author has a longer name.

Ticket No: #58844

Trac ticket:

ifaisaldev commented on PR #5714:


5 months ago
#9

Fix this issue on mobile "Restore This Revision" button in the wp-admin revision page can become hidden behind the diff panel in mobile view, particularly if the post author has a longer name.

Trac ticket: https://core.trac.wordpress.org/ticket/58844

Note: See TracTickets for help on using tickets.