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.

Sign up for GitHub

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

Jump to bottom

Add option to export altered (rotated, etc.) versions of photos from iPhoto #1396

Closed
mlevin77 opened this issue Jan 22, 2024 · 3 comments
Closed

Add option to export altered (rotated, etc.) versions of photos from iPhoto #1396

mlevin77 opened this issue Jan 22, 2024 · 3 comments
Labels
feature request New feature or request

Comments

@mlevin77
Copy link

Right now, it exports the original photos. iPhoto offers ability to rotate (and otherwise edit, but rotation is the most important one) the photos. Apparently they are stored separately, and not actually over-writing the original files. I'd love there to be an option to export the rotated ones, instead of the originals.
@mlevin77 mlevin77 added the feature request New feature or request label Jan 22, 2024
@RhetTbull
Copy link
Owner

If a photo is rotated in iPhoto the following happens:

photo.hasadjustments is False (assuming no previous adjustments)
photo.width, photo.height are adjusted accordingly
photo.orientation does not change (is 0 for iPhoto)
photo._info["rotation"] is set to the degrees rotated (e.g. 90 deg counter clockwise = 270]

RhetTbull added a commit that referenced this issue Feb 25, 2024
@RhetTbull
Implemented --fix-orientation, #1396
cb635d5
@RhetTbull
Copy link
Owner

Implemented as --fix-orientation. Need to add tests.

RhetTbull added a commit that referenced this issue Feb 25, 2024
@RhetTbull
Implemented --fix-orientation, #1396
112de8d
@RhetTbull RhetTbull mentioned this issue Feb 25, 2024
Merged
RhetTbull added a commit that referenced this issue Feb 25, 2024
@RhetTbull
Iphoto rotation 1396 (#1414)
a45a934 
* Updated tests for rotated photo

* Implemented --fix-orientation, #1396

* Implemented --fix-orientation, #1396

* Added test
@RhetTbull
Copy link
Owner

This is included in release v0.67.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mlevin77 @RhetTbull