ailon / cropro

Crop, rotate, flip, and straighten images in your web apps.
Other
13 stars 4 forks source link

iOS bug around resizing when switching from portrait to landscape back to portrait #7

Closed tmigacz closed 1 year ago

tmigacz commented 3 years ago

This bug is specific only for iOS phones, and only when using Safari's "Add to Home Screen" feature. Note that the website needs to a web app manifest file to support the "Add to Home Screen" feature.

When using the cropping tool in portrait mode on your iOS phone, when you switch to landscape view, the height of the cropping tool is updated, but when you switch the phone back to portrait view, the height is not updated back.

Steps to reproduce:

Here's what it looks like when the height isn't properly updated (note half the screen is not covered by the cropping tool):

image

ailon commented 3 years ago

Hey. Thank you for reporting this. I don't have access to an iOS device right now, will try to reproduce later.

What is that underlying page? It looks like the whole thing isn't resized either. Or does it just end there?

tmigacz commented 3 years ago

Thanks for the quick response!

The page just ends there. We tried it on a larger page with more content and it's only the cropping (and maker tool) that doesn't get resized.

ailon commented 3 years ago

Thank you for the details. I'll try to replicate this, but if it only happens when adding to home screen, it feels like a Safari bug. This shouldn't be any different than a normal browser window from the code's perspective.

In any case, feels like this is going to be painful to debug without a Mac :/

tmigacz commented 3 years ago

Agreed, but I still appreciate you taking a look.