WordPress.org

Make WordPress Core

Opened 6 months ago

Last modified 6 months ago

#46971 new defect (bug)

Selected Image gets overflow in Edit Image in Media upload - iPhone - 6/7/8

Reported by: shashank3105 Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Media Keywords: has-patch needs-testing has-screenshots dev-feedback
Focuses: Cc:
PR Number:

Description

Hello,

While we edit image from admin dashboard (Dashboard > Media > Library - edit), the selected image gets overflow in iPhone-6/7/8 portrait mode. So for that, we can apply max-width: 100%.

Thanks,
Shashank.

Attachments (3)

46971-shashank3105.patch (395 bytes) - added by shashank3105 6 months ago.
Patch file for this issue
iPhone 6_7_8-before-change.png (199.2 KB) - added by shashank3105 6 months ago.
Screenshot for issue
iPhone 6_7_8-after-change.png (186.9 KB) - added by shashank3105 6 months ago.
Screenshot for solved issue

Download all attachments as: .zip

Change History (8)

@shashank3105
6 months ago

Patch file for this issue

@shashank3105
6 months ago

Screenshot for issue

@shashank3105
6 months ago

Screenshot for solved issue

This ticket was mentioned in Slack in #core-media by joemcgill. View the logs.


6 months ago

#2 @joemcgill
6 months ago

  • Keywords dev-feedback added

Hi @shashank3105,

Thanks for the report and providing a patch. However, I'm having trouble reproducing this in the latest 5.2-beta3 version of WordPress, so perhaps this has already been fixed?

#3 @shashank3105
6 months ago

Hi @joemcgill ,

Thank you for your feedback.

I am generating this issue in the development version (5.2-beta3-45248). So can you please try to regenerate this issue by the following step?

From Admin Dashboard > Media > Library > Select edit option of previously uploaded image > After that click on Edit Image button underneath image. Please check attached video link for the generated issue.

https://drive.google.com/open?id=1bMoThUEagy88G2Hd_d6AAHDs42i6xBz5

Thank you.

#4 @SergeyBiryukov
6 months ago

Hi @shashank3105, thanks for the ticket and the patch!

Removing the trunk version, as this doesn't seem to be a regression in 5.2.

#5 @SergeyBiryukov
6 months ago

  • Version trunk deleted
Note: See TracTickets for help on using tickets.