site-menu.js: have lit element load sites from the server

Currently the lit element for the site menu has to load in its entirety with all of the user’s sites but this can cause the page to load slowly and may hinder the user from completing other desired actions other than selecting a site.

We should change the lit element to load the sites from the server like it used to in the javascript menu that the lit element replaced.

I (Trek) wanted to have everything load together but given that there is a noticeable delay in loading due to processing, I think it’s best to load in parts (lazy loading).

We must make sure that there in no false information being shown, eg. an empty sitemenu, in the meantime while it loads because this could lead to confusion. Instead we should have an indicator that the menu is loading eg. a label that says “loading…”.

Comments (1)