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

clickHandler not firing in IE 11 #420

Closed
philipgiuliani opened this issue Jun 16, 2014 · 11 comments
Closed

clickHandler not firing in IE 11 #420

philipgiuliani opened this issue Jun 16, 2014 · 11 comments
Labels

Comments

@philipgiuliani
Copy link
Contributor

The clickHandler is not firing in Internet Explorer 11. IE 9 works as expected and IE 10 isn't tested.

I also opened an issue at the ViewerInputHook repository, but i think it could also be because of the mousetracker, so i also post it here.

msalsbery/OpenSeadragonViewerInputHook#3

@msalsbery
Copy link
Member

It's indeed in MouseTracker and a fix is in the works.

@iangilman iangilman added the bug label Jun 16, 2014
@philipgiuliani
Copy link
Contributor Author

Thanks, already any release date? We are using this amazing project on a bigger website (http://epaper.e-dicola.net) and got already a lot of mails that the overlays are not clickable in IE 11 :(

@msalsbery
Copy link
Member

I'll submit a patch today or tomorrow...I've been lagging, sorry.

The problem is identified, just needs a cross-browser solution which has been elusive :)

@iangilman
Copy link
Member

@philipgiuliani wow, epaper is beautiful! Is each issue a single giant DZI?

@philipgiuliani
Copy link
Contributor Author

Thanks a lot :) Yes they are very big! There is also a magazine with 640 Pages. Im joining the pages and generating the Deepzoomimage with libvips :)

@iangilman
Copy link
Member

Very cool. Seems to work well. For the future, you may be interested in #386.

@philipgiuliani
Copy link
Contributor Author

Yes i already bookmarked this Issue a long time a go and im just waiting for it :) Thanks!

@andreasunterhuber
Copy link

Hi msalsbery! Anything new about the fix? Thanks

@msalsbery
Copy link
Member

@philipgiuliani @keepinmind

I finally have a new patch for this, and could use some external testing!.

Please test at this demo site

OpenSeadragon builds to test with:
openseadragon.js
openseadragon.min.js

@msalsbery
Copy link
Member

Fixed by #448

@philipgiuliani
Copy link
Contributor Author

Hi sorry for the late answer. We updated it yesterday on epaper.e-dicola.net and it seems to work in all browsers. Thanks!

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

No branches or pull requests

4 participants