8000 Crop should mirror orientation changes · Issue #17498 · darktable-org/darktable · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Crop should mirror orientation changes #17498

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
Closed
emko opened this issue Sep 19, 2024 · 3 comments
Closed

Crop should mirror orientation changes #17498

emko opened this issue Sep 19, 2024 · 3 comments
Labels
difficulty: hard big changes across different parts of the code base duplicate feature: enhancement current features to improve no-issue-activity scope: UI user interface and interactions
Milestone
5.2

Comments

@emko
Copy link
Contributor
emko commented Sep 19, 2024 •
edited
Loading

Is your feature request related to a problem? Please describe.
I have set a shortcut to flip my image while editing so i can switch back and forth quickly, its something i have done with drawing,painting etc to see the image in a different view, the issue is if there is any cropping applied when doing any orientation changes the crop does not follow the orientation changes so you have to edit the crop.

Describe the solution you'd like
when changing any of the orientation settings the crop should be adjusted to match, kind of like how every other software does it there shouldn't be a need to readjust your crop anytime you rotate/flip etc your image

thanks
@ralfbrown ralfbrown added duplicate feature: enhancement current features to improve difficulty: hard big changes across different parts of the code base scope: UI user interface and interactions labels Sep 20, 2024
@ralfbrown
Copy link
Collaborator
ralfbrown commented Sep 20, 2024 •
edited
Loading

Duplicate of #11614 and #14788.

TL;DR: move orientation above crop if you want to use it for checking different views.

@emko
Copy link
Contributor Author
emko commented Sep 20, 2024 •
edited
Loading

Duplicate of #11614 and #14788.

TL;DR: move orientation above crop if you want to use it for checking different views.

sorry, i though i did ask i tried searching for already existing issue, should i remove this one?

also you can't move orientation module. I tried other modules to see and retouch and liquify understands when orientation changes it follows but vignetting module does not and behaves the same as crop

@ralfbrown ralfbrown mentioned this issue Oct 9, 2024
Closed
@github-actions GitHub Actions
Copy link

This issue has been marked as stale due to inactivity for the last 60 days. It will be automatically closed in 300 days if no update occurs. Please check if the master branch has fixed it and report again or close the issue.

@jenshannoschwalm jenshannoschwalm mentioned this issue Nov 25, 2024
Merged
@jenshannoschwalm jenshannoschwalm added this to the 5.2 milestone Nov 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
difficulty: hard big changes across different parts of the code base duplicate feature: enhancement current features to improve no-issue-activity scope: UI user interface and interactions
Projects
None yet
Milestone
5.2
Development

No branches or pull requests
3 participants
@emko @ralfbrown @jenshannoschwalm
0