Skip to content

Conversation

danzel
Copy link
Member

@danzel danzel commented Apr 30, 2013

Fixes #1539

@mourner
Copy link
Member

mourner commented Apr 30, 2013

Thanks for the fix Dave! Can you explain the fix btw? I'm not sure what's happening here :)

mourner added a commit that referenced this pull request Apr 30, 2013
Fix layers control on WinPhone8/IE10 Touch
@mourner mourner merged commit 47dd2d9 into Leaflet:master Apr 30, 2013
@danzel
Copy link
Member Author

danzel commented Apr 30, 2013

IE10 generates a mouseover event when you put your finger down and a mouseout event when you release, this is what was breaking the layers control before.

This clued me in on on the fix:
http://www.cssplay.co.uk/menus/cssplay-droplist-menu-ie10-touch-screen.html
http://wordpress.org/support/topic/ie10-ie8-on-touch-devices

Setting aria-haspopup somehow makes the events work right, I'm not totally sure what it changes, I think it makes it stop firing the mouseover event when you release your initial touch (I stopped investigating when it worked).

@mourner
Copy link
Member

mourner commented Apr 30, 2013

Allright. :) I think it's worth adding a comment to the fix then, as it's not obvious (you can commit this directly without a pull).

@Malvoz Malvoz added the ie Internet Explorer label May 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ie Internet Explorer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Layer control does not work on Windows Phone 8 (Lumia 820)
3 participants