-
Notifications
You must be signed in to change notification settings - Fork 1
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
Comments
@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? |
@ebgoldstein I think the first two are best. The first one is nice cause the zoomed in portion follows the mouse 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
I don't think the scribble will cause any issues though, but thats the only thing I can see going wrong. |
thx for the guidance @ShahNafisRafique ! |
e.g.,
https://www.npmjs.com/package/react-cursor-zoom
https://www.npmjs.com/package/react-image-zoom
https://www.npmjs.com/package/react-inner-image-zoom
The text was updated successfully, but these errors were encountered: