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

Wrong zoomer position on hover #86

Closed
DreaMinder opened this issue Oct 17, 2018 · 6 comments · Fixed by #113
Closed

Wrong zoomer position on hover #86

DreaMinder opened this issue Oct 17, 2018 · 6 comments · Fixed by #113

Comments

@DreaMinder
Copy link

Problem
At first moment when zoomer appears, zoomed image inside has wrong position or invisible. When cursor moves along, zoomer works fine.

Reproduce

  1. Go to https://juliapomoshko.com/en/store/coat-grey-long-with-belt
  2. Hover your cursor over image with a fast move and stop.

Additional context
I use drift-zoom inside of vue.js component. Version with the issue is 1.3. And 1.2 works fine (but has iOS bug, so I can't use it instead).

@frederickfogerty
Copy link
Contributor

Hey @DreaMinder thanks for the bug report! We've noticed this internally as well. I'll look to get onto this either this week or next.

@frederickfogerty
Copy link
Contributor

Hey @DreaMinder I believe I've fixed this in #113. If you'd like to give this a review, I'd be more than happy to take any feedback you have!

@DreaMinder
Copy link
Author

@frederickfogerty hi. It doesn't build on windows =(
But I'm sure you got this, looking forward for release
Thanks for awesome package, btw!

@frederickfogerty
Copy link
Contributor

@DreaMinder would it help if I sent you a compiled version of the dist folder?

@DreaMinder
Copy link
Author

@frederickfogerty I guess so, drop it here dreaminder7@gmail.com, I'll test it

@DreaMinder
Copy link
Author

@frederickfogerty oh nevermind, I already installed new release, works like a clockwork!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants