Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Portrait Photos loaded in Landscape Orientation #85

Open
peter-genesys opened this issue Jul 13, 2021 · 0 comments
Open

Portrait Photos loaded in Landscape Orientation #85

peter-genesys opened this issue Jul 13, 2021 · 0 comments

Comments

@peter-genesys
Copy link

Expected behavior

Portrait Photos loaded into Dropzone remain in portrait orientation

Actual behavior

Photos taken with some phones, and loaded without resizing or compression, are loaded as landscape in the thumbnail, and also in the loaded file.
Examples include iPhone 6S (Apple) and Samsung A8 (android)

Workaround

Before loading a photo, resize or recompress it with tools like TinyPNG (website) or Lit Photo (android app).
This seems to lock the orientation.

Steps to reproduce the issue

Take a portrait photo with phone.
Do NOT modify the photo in any way, either by cropping, resize or recompression.
Upload photo directly from camera or from photos on phone filesystem into APEX Dropzone plugin.
Thumbnail loads in landscape orientation.
Result photo is also in landscape orientation.
Example photo attached.
20210714_083249

APEX version (Oracle APEX 20.2 with Oracle DB XE 18c)

Used web server / version and platform (ORDS 20.3 on Cent OS 7)

Used web browser / version and platform (Chrome Version 91.0.4472.124 Windows)

Cause

The underlying project dropzone version is 5.5.0
This version has a bug in ExifRestore.restore that causes the rotation.

Quick Fix

One fix is to comment out the call to ExifRestore.restore.
A better solution is to update the underlying dropzone code to the latest dropzone version, which includes another fix to Exif, which I assume fixes the bug.

I will open a pull-request on this project with my Quick Fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant