Page MenuHomePhabricator

Add a background image with a spinner or similar to static maps
Open, Needs TriagePublic

Description

To make it more visible to users that loading is happening, we should display a visual information. This can be a spinner or something else. See T305433#7920365 for some examples.

Screen Shot 2022-04-20 at 17.57.02.png (372×501 px, 11 KB)
Screen Shot 2022-04-19 at 15.07.05.png (416×415 px, 22 KB)

Event Timeline

Really have to separate static and dynamic maps on this issue.

Specifically, for dynamic maps, I think that a big issue currently is, that the data layers need to finish downloading before some of the map setup (show the dialog) is completed. That doesn't really make sense, we should load the map before downloading an adding the additional layers. This is especially noticeable on something like the governor/state map on https://www.mediawiki.org/wiki/Help:Extension:Kartographer#/map/4

It might be good to decouple those dependencies and make sure that the ready() callback of the map is sent before the data layers start. Or maybe there should just be mutliple events ? loading/ready etc.