WordPress.org

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#28592 closed defect (bug) (fixed)

Post edit screen goes out of view port in smaller devices

Reported by: vinod dalvi Owned by: ocean90
Milestone: 4.0 Priority: normal
Severity: normal Version: 3.9.1
Component: Editor Keywords: has-patch commit
Focuses: ui Cc:
PR Number:

Description

When we try to access edit post screen in between the following browser window size then it goes out of window.

Greater than:
Window size: 917 x height
View port size: 901 x height

Less than:
Window size: 977 x height
View port size: 961 x height

See the attached screenshot.

Patch added.

Attachments (5)

Edit Post ‹ WPMU Site — WordPress.jpg (126.6 KB) - added by vinod dalvi 5 years ago.
Screenshot of the issue.
28592.diff (1020 bytes) - added by vinod dalvi 5 years ago.
Patch File.
28592-2.diff (507 bytes) - added by vinod dalvi 5 years ago.
Patch added in edit.css file.
28592-3.diff (489 bytes) - added by vinod dalvi 5 years ago.
Made some minor improvements in the patch.
28592.patch (1.0 KB) - added by ocean90 5 years ago.

Download all attachments as: .zip

Change History (14)

@vinod dalvi
5 years ago

Screenshot of the issue.

@vinod dalvi
5 years ago

Patch File.

#1 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 4.0
  1. This should probably go into wp-admin/css/edit.css, since it's not specific to the admin menu.
  2. No need to patch admin-menu-rtl.css, it's generated automatically.

@vinod dalvi
5 years ago

Patch added in edit.css file.

#2 @vinod dalvi
5 years ago

  • Keywords has-patch added

@vinod dalvi
5 years ago

Made some minor improvements in the patch.

#3 @DrewAPicture
5 years ago

  • Keywords commit added

28592-3.diff fixes it for me at those viewport sizes.

This ticket was mentioned in IRC in #wordpress-dev by DrewAPicture. View the logs.


5 years ago

This ticket was mentioned in IRC in #wordpress-dev by DrewAPicture. View the logs.


5 years ago

#6 follow-up: @helen
5 years ago

  • Keywords commit removed

Should we just autofold the admin menu a little earlier then? I'm not really excited about adding a viewport rule just for this.

@ocean90
5 years ago

#7 in reply to: ↑ 6 @ocean90
5 years ago

  • Keywords commit added

Replying to helen:

Should we just autofold the admin menu a little earlier then?

Done in 28592.patch.

#8 @ocean90
5 years ago

  • Owner set to ocean90
  • Resolution set to fixed
  • Status changed from new to closed

In 29518:

Admin Menu: Auto fold the menu at 960px.

This gives us some more room at the post edit screen.

props vinod dalvi for initial patch.
fixes #28592.

#9 @helen
5 years ago

#28304 was marked as a duplicate.

Note: See TracTickets for help on using tickets.