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

Overlays don't get horizontally fliped when the image it's flipped #2450

Open
Zardoz89 opened this issue Jan 10, 2024 · 4 comments
Open

Overlays don't get horizontally fliped when the image it's flipped #2450

Zardoz89 opened this issue Jan 10, 2024 · 4 comments

Comments

@Zardoz89
Copy link

So, you open an image with an overlay that draws a rectangle over it. If using the OSD intrface, an user flips horizontally the image, the overlay don't get flipped. At least, it's strange, specially when it works correctly using the rotation buttons. Here is a example :

Original image + overlay :
image

Flipped image + overlay :
image

@iangilman
Copy link
Member

Indeed, this is a known issue: #1614

Evidently no one has decided to take it on. Let me know if you are interested in working on it and I can point you in the right direction.

@BeebBenjamin
Copy link

I need this fixed at some point too. I can attempt to fix it under guidance.

@Zardoz89
Copy link
Author

Also, I could try to help with it.

@iangilman
Copy link
Member

@BeebBenjamin @Zardoz89 Fantastic! Presumably the fix will go into this function:

_getOverlayPositionAndSize: function(viewport) {

You can probably get some inspiration from the rotation handling.

I expect the typical user will want the overlay position flipped but not the actual overlay contents flipped (i.e. they wouldn't want the overlay text to be backwards). Though I could see an argument for the other way. What do you think?

I'll leave it to you to figure out who is taking the lead on this.

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