mapbox / mapbox-studio-classic

https://www.mapbox.com/mapbox-studio/
BSD 3-Clause "New" or "Revised" License
1.14k stars 229 forks source link

Changing sources breaks mouse input in Chrome #1291

Closed TheMapSmith closed 9 years ago

TheMapSmith commented 9 years ago

Hello,

For some reason when I add my remote source to the Basic Map template, MBS just sits there doing nothing. No spike in processing/memory/network activity. No error message. The page just becomes unresponsive.

Process sample

I'm able to add the source to a blank map, but as soon as I try to integrate it with mapbox-streets-v5 it hangs.

Update: Starting a new project and not worrying about my GeoJSON source now. Here are the steps I took:

  1. Start a new project with the Basic style
  2. Change source to Mapbox Streets + Vector Terrain
  3. Upon clicking Apply no more mouse inputs are accepted. Keyboard shortcuts continue to work.

Tried to confirm this behavior in Firefox and Safari, but they both work just fine.

Tried it again in Incognitio with extensions disabled and the problem still occurs in Chrome.

TheMapSmith commented 9 years ago

This was fixed with either a fresh clone and install or one of the updates.

samanpwbb commented 9 years ago

Great, :+1: - pretty sure this was a webkit bug affecting our modal, where the element was visually hidden but still occupying space on the page.