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

[FR] rotate 90° / orientation tag #129

Closed
1 task done
ildar opened this issue Sep 10, 2018 · 11 comments
Closed
1 task done

[FR] rotate 90° / orientation tag #129

ildar opened this issue Sep 10, 2018 · 11 comments

Comments

@ildar
Copy link

ildar commented Sep 10, 2018

the app is editing EXIF but doesn't handle the most used tag: Orientation.
Please implement lossless rotation 90°/180°/270° through EXIF. As an UI example see Shotwell application.
Thanks

@k3b k3b added this to the 0.7.2 milestone Sep 10, 2018
k3b added a commit that referenced this issue Oct 17, 2018
…in com.github.k3b:PhotoView:v1.2.6.k3b-7-SNAPSHOT
@k3b
Copy link
Owner

k3b commented Oct 24, 2018

fixed in 0.7.2

@k3b k3b closed this as completed Oct 24, 2018
@ildar
Copy link
Author

ildar commented Apr 29, 2019 via email

@k3b
Copy link
Owner

k3b commented Apr 29, 2019

Sorry my fault: i did not read the complete question.

I only fixed display in wrong orienentation when orientation flag was set.

but not implemented activly-modify the orientation in the exif from the gui = edit

@ildar
Copy link
Author

ildar commented Apr 29, 2019 via email

@k3b k3b reopened this Apr 29, 2019
@supercub
Copy link

supercub commented Oct 6, 2019

Hi, i'm using "A Photo Manager (Version AndroFotoFinder 0.7.5.190916)" and even if orientation flag is set in a picture, it isn't rotated automatically after clicking on it. At the preview grid it is displayed correct, but not on in detail view. The EXIFTOOL output of this picture is 'Orientation: Rotate 90 CW'.
See also #148. My camera is a Casio Exilim HS.

@ildar
Copy link
Author

ildar commented Oct 28, 2019 via email

@supercub
Copy link

Hi k3b, thanks for fixing the bug. In version 0.8.0 pics are now rotated automatically according to their EXIF orientation flag.

@k3b
Copy link
Owner

k3b commented Oct 29, 2019

@supercub thanks for feedbak

@k3b
Copy link
Owner

k3b commented Oct 29, 2019

@ildar sorry, my fauld again: :-( . i have created a new ticket for "command to rotate image" in #154 which i hopefully will not misinterprete again as "please fixe the wrong image rotation bug".

@ildar
Copy link
Author

ildar commented Oct 30, 2019 via email

@k3b
Copy link
Owner

k3b commented Nov 6, 2019

the next llcrop version will support image rotation https://github.com/k3b/LosslessJpgCrop/ . you can call it a a photo editor from a photo manager.

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

No branches or pull requests

3 participants