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.

Sign up for GitHub

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

Jump to bottom

Navigation Control compass click event IE 11 throws: Object doesn't support this action #4784

Closed
keyofj opened this issue Jun 5, 2017 · 1 comment
Closed

Navigation Control compass click event IE 11 throws: Object doesn't support this action #4784

keyofj opened this issue Jun 5, 2017 · 1 comment
Labels
bug 🐞 environment-specific 🖥️

Comments

@keyofj
Copy link

keyofj commented Jun 5, 2017

mapboxgl - v.0.37.0
Internet Explorer - 11.0.38

When you click the compass the following error is thrown only in Internet Explorer:

Object doesn't support this action
File: mapbox-gl.js, Line: 370, Column: 41

If you click on the error it takes me to line 105 in function copyMouseEvent(e)

This can be duplicated using the examples page: Navigation Control

Open IE Debugger Tools (F12) and you will see the error above.

In the example page, the map will go ahead and return to due North, but in applications that use IE 11 embedded the JavaScript error raises an alert window and will not allow the application to continue.
@anandthakker anandthakker added bug 🐞 cross-platform 📺 Requires coordination with Mapbox GL Native (style specification, rendering tests, etc.) labels Jun 6, 2017
@andrewharvey andrewharvey mentioned this issue Jun 16, 2017
Closed
5 tasks
@1ec5 1ec5 added environment-specific 🖥️ and removed cross-platform 📺 Requires coordination with Mapbox GL Native (style specification, rendering tests, etc.) labels Jun 20, 2017
jfirebaugh added a commit that referenced this issue Aug 17, 2017
@jfirebaugh
Reimplement compass using DragRotate handler
f9ec635 
Reuse the existing code directly, rather than synthesizing mouse events, which doesn't work on IE (#4784) and triggers mouse event listeners unexpectedly (#5148).
@jfirebaugh jfirebaugh mentioned this issue Aug 17, 2017
Merged
jfirebaugh added a commit that referenced this issue Aug 17, 2017
@jfirebaugh
Reimplement compass using DragRotate handler
12f2662 
Reuse the existing code directly, rather than synthesizing mouse events, which doesn't work on IE (#4784) and triggers mouse event listeners unexpectedly (#5148).
jfirebaugh added a commit that referenced this issue Aug 22, 2017
@jfirebaugh
Reimplement compass using DragRotate handler
4a5233c 
Reuse the existing code directly, rather than synthesizing mouse events, which doesn't work on IE (#4784) and triggers mouse event listeners unexpectedly (#5148).
jfirebaugh added a commit that referenced this issue Aug 22, 2017
@jfirebaugh
Reimplement compass using DragRotate handler
b203a40 
Reuse the existing code directly, rather than synthesizing mouse events, which doesn't work on IE (#4784) and triggers mouse event listeners unexpectedly (#5148).
@jfirebaugh
Copy link
Contributor

Fixed in #5157.

@jmunowitch jmunowitch mentioned this issue Sep 14, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐞 environment-specific 🖥️
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

include MouseEvents polyfill to allow map rotation via compass button on IE11 mapbox/mapbox-gl-js
4 participants
@jfirebaugh @1ec5 @anandthakker @keyofj