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

Increazing zoom doesn't zoom at center of current view #30

Open
ExtReMLapin opened this issue Nov 8, 2020 · 3 comments
Open

Increazing zoom doesn't zoom at center of current view #30

ExtReMLapin opened this issue Nov 8, 2020 · 3 comments
Assignees
Labels
3.1.x 🦌 enhancement Improve existing feature 🥇 good first issue Good for newcomers

Comments

@ExtReMLapin
Copy link

ExtReMLapin commented Nov 8, 2020

When you press CTRL+Mouse Wheel

Example :

Current pix with resolution that doesn't matter is zoomed and rendered at resolution of 500x500

Picture is centered so zoom center is at 250x250

You zoom with ctrl+Mouse Wheel

new render resolution is 600*600 but it sets center to 250*250 instead of 300*300

video https://www.youtube.com/watch?v=SvjfN36HtNE&feature=youtu.be

@Beelink Beelink self-assigned this Nov 8, 2020
@Beelink Beelink added 🏭 3.x.x To implement in v3.0.0 and later 🦌 enhancement Improve existing feature 🥇 good first issue Good for newcomers labels Nov 8, 2020
@Beelink Beelink added 3.1.x and removed 🏭 3.x.x To implement in v3.0.0 and later labels Mar 24, 2021
@ueen
Copy link

ueen commented May 6, 2022

Yea the zoom is always anchored to the top left, i would love to have a mouse-centered zoom (thats the only thing the built in photos app does better currently) - otherwise just awesome app, so much clever features i absolutly need, now that i know they exist <3

@ExtReMLapin
Copy link
Author

It's actually pretty easy to do, I had to do it multiple times with canvas on web applications.

The idea at the end of the day is to be sure your mouse is at the same relative position on your image (in percentage, not in pixel).

@ueen
Copy link

ueen commented May 8, 2022

Awesome, are you maybe able to do a PR? @ExtReMLapin 😍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.1.x 🦌 enhancement Improve existing feature 🥇 good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants