search

fat / zoom.js

Medium's Image Zoom for jQuery
https://fat.github.io/zoom.js
MIT License
4.04k stars 332 forks source link

Append element to document body instead of wrapping the photo #77

Closed ElegantSudo closed 8 years ago

ElegantSudo commented 8 years ago

There should be an easy and performant way to append the wrapper with the image to the document body, with the exact positioning of the previous photo (positioned absolute to body, top and left values).

This would solve the overflow: hidden a lot of users are currently having because the wrapper would be positioned relative to the body element.

ElegantSudo commented 8 years ago

@fat what are your thoughts on this? Maybe there's a smarter way to design our apps or work around this issue.

ElegantSudo commented 8 years ago

@fat hey, I'm not sure if you saw my post up there. Let me know what you think.

ElegantSudo commented 8 years ago

@alisonmonteiro what are your thoughts on this issue?

ElegantSudo commented 8 years ago

Fixed in forked repo. This repo is unmaintained garbage now.

Ilshidur commented 7 years ago

Can you link the forked repo please ?

ElegantSudo commented 7 years ago

@Ilshidur it's linked above in the merged commit by spinningarrow.

Ilshidur commented 7 years ago

Thank you !