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

Hover zoom #193

Open
ebgoldstein opened this issue Mar 21, 2021 · 3 comments
Open

Hover zoom #193

ebgoldstein opened this issue Mar 21, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@ebgoldstein
Copy link
Contributor

@ebgoldstein ebgoldstein added the enhancement New feature or request label Mar 21, 2021
@ebgoldstein
Copy link
Contributor Author

@ShahNafisRafique - asking for your opinion - do you think this is an easy thing to integrate? and do you have any insight into which implementation might be best for this project?

@ShahNafisRafique
Copy link
Contributor

ShahNafisRafique commented Mar 22, 2021

@ebgoldstein I think the first two are best.

The first one is nice cause the zoomed in portion follows the mouse
The second is nice cause it shows what part of the image is zoomed in with the grey part
The third one will probably mess up with the drawing tool too much.

If I had to pick one, I would probably say the 1st since it would require slightly less UI changes/thinking as the second would require some area to be designated to show the zoomed in portion

The 2 hardest things to do are

  1. get the image size (which is already done) and
  2. making it work with the scribble.

I don't think the scribble will cause any issues though, but thats the only thing I can see going wrong.

@ebgoldstein
Copy link
Contributor Author

thx for the guidance @ShahNafisRafique !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants