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

Cannot read property 'removeEventListener' #7

Open
benjamin-sln opened this issue Feb 16, 2018 · 8 comments
Open

Cannot read property 'removeEventListener' #7

benjamin-sln opened this issue Feb 16, 2018 · 8 comments

Comments

@benjamin-sln
Copy link

ZoomableImage._this.removeBodyClickHandlers fail when unmount component without zooming on it.

@connorvincent
Copy link

In node_modules\react-zoomable-image\build\index.js, put if(_this.bodyElement) above line 857.
line 857: (_this$bodyElement2 = _this.bodyElement).removeEventListener.apply(_this$bodyElement2, _toConsumableArray(handler));

@LeeMin-hye
Copy link

same problem here:(
it doesn't work with "put if(_this.bodyElement) above line 857".
any idea?
isn't it bug??
cap1
cap2

@LeeMin-hye
Copy link

i just figure out this problem!!!!!
In node_modules\react-zoomable-image\build\index.js, put if(_this.bodyClickHandlers) above line 854!
line 854: _this.bodyClickHandlers.forEach(function (handler)

hopefully, i'd like to save someone who got the same matter like me:)
Gd luck!

@connorvincent
Copy link

I decided to use react-image-magnify

@LeeMin-hye
Copy link

"react-image-magnify" looks good//
It's a little bit different what I want to use, though.
I was looking for the itself can zoom in to where no needs to have other area..
i thnk zoomable-image more fits me...
anwy nxt time, i will try using react-image-magnify!

@connorvincent
Copy link

I was looking for the itself can zoom in to where no needs to have other area..

It supports this. Check out their demo: https://ethanselzer.github.io/react-image-magnify/#/react-slick
The location and size of an enlarged image can also be changed

@LeeMin-hye
Copy link

WOW!
it's exactly what i wnt!

i dnt understand why i didn't find this before t.t

anwy, thanks a lot!!!!!!!!!
u a my hero;)

@LeeMin-hye
Copy link

i tried react-image-magnify..bt it doesn't work with my project...

i used react-simple-zoom!
it's very simple..
gd fr me haha//

just in case, sharing the site:
https://www.npmjs.com/package/react-simple-zoom

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

No branches or pull requests

3 participants