Make WordPress Core

Opened 3 years ago

Last modified 3 years ago

#58844 new defect (bug)

Revisions: Compare Revisions layout issues on mobile

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

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 3 years ago.

Download all attachments as: .zip

Change History (10)

#1 @adamsilverstein
3 years ago

  • Milestone Awaiting Review → 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
3 years ago

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

#3 @adamsilverstein
3 years ago

  • Version 6.2.2 → 3.6

#4 @plari
3 years 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
3 years 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.


3 years 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:


3 years 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.