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.

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

Always let user know if page is still loading or not #10814

Open
jidanni opened this issue Feb 24, 2025 · 6 comments
Open

Always let user know if page is still loading or not #10814

jidanni opened this issue Feb 24, 2025 · 6 comments
Labels
bug-cant-reproduce Having trouble reproducing this issue

Comments

@jidanni
Copy link
Contributor

jidanni commented Feb 24, 2025

Rather often this week, everything works fine, except there is no map data:
Image
I checked DevTools network tab and there were no errors there.
I do only have a 2M/64K ADSL connection, but that shouldn't be the culprit.
And yes, this is a heavily mapped area.

@tyrasd
Copy link
Member

tyrasd commented Feb 25, 2025

Oh, that's pretty odd. Something must have gone wrong while loading the map data…

I checked DevTools network tab and there were no errors there.

Are there also no errors in the developer tools console tab?

@tyrasd tyrasd added the bug-cant-reproduce Having trouble reproducing this issue label Feb 25, 2025
@jidanni jidanni closed this as not planned Won't fix, can't repro, duplicate, stale Feb 25, 2025
@jidanni
Copy link
Contributor Author

jidanni commented Feb 25, 2025

Here it happened again, after an edit.

Image

The console just has a few [Violation] 'requestIdleCallback' handler took 59ms
Clicking the OSM logo to leave iD, then clicking Edit to return, doesn't help.

Closing the browser tab, opening a new one and navigating to the site doesn't help.

Nope. It seems all I can do is a couple edits a day before this starts happening. I bet it is due to a heavily loaded system causing some items to timeout and never load.

@tyrasd tyrasd reopened this Feb 26, 2025
@tyrasd
Copy link
Member

tyrasd commented Feb 26, 2025

Does switching to a different browser maybe help? If you can, also trying a different network connection might be worth a try.

One more information to help debug the problem would be to check what the OSM API returns when this happens. Go to the network tab and click on one of the map.json calls to see what the response looks like. It should look like in the screenshot below (Chrome, but Firefox should be quite similar), except that in a working case the elements would not be an empty list ([]) in a well mapped region:

Image

@jidanni
Copy link
Contributor Author

jidanni commented Feb 26, 2025

Bummer.

There are all kinds of *.json...

Image

Just no map.json!

Kind of makes one feel bad.

P.S., it's ChromeOS, so let's debug this for Chrome. P.S.S., those blue streaks above are when the background imagery finally loaded. I.e., I'll keep my system heavily loaded until we can find out what messes up iD in such situations.

@jidanni
Copy link
Contributor Author

jidanni commented Feb 26, 2025

Wait, sometimes there is partial data,

Image

OK, I started DevTools and hit CTRL+R, and alas the data went away.

Image
is the only hint of any map.json. Console: same as previous mention.

@jidanni
Copy link
Contributor Author

jidanni commented Feb 26, 2025

Opinion:

No spinners or anything anywhere to tell the user the page is still loading.

  • No spinner seen before imagery loads. User just stares at black screen.

  • No spinner seen before map data loads. User just stares at pretty imagery as if the place has never been mapped yet.

  • And of course no old fashioned browser built-in spinners, because this is no simple HTML document.

How to reproduce: try iD on a third world network with a third world laptop.

No I will never know if in fact I am still waiting for something to load, as even the bottom of the page,

Image

Gives no clue.

How about putting "Page not fully loaded yet" there, and only clearing it out when all the stuff has finally loaded:

Image

(I recall there used to be a spinner down there last year.)

P.S., don't freak out about the "hidden features" above. Yes, I hid all but roads.

Anyway, let's get those indicators of page loading status into the code. Then we will be able to debug such problems further.

(OK, #8373 ialso ndicates I should also test on another browser. OK, I installed chromium (yes, rhymes with chrome) and indeed my craptop became a CRAY and iD acted like it was on Vi*g*a, no bugs! But that's beside the point.)

But still iD should always have a spinner...

@jidanni jidanni changed the title No map data Always let user know if page is still loading or not Feb 26, 2025
@jidanni jidanni marked this as a duplicate of #10163 Feb 26, 2025
@jidanni jidanni marked this as a duplicate of #7688 Feb 26, 2025
@jidanni jidanni marked this as a duplicate of #8444 Feb 26, 2025
@jidanni jidanni marked this as a duplicate of #7451 Feb 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-cant-reproduce Having trouble reproducing this issue
Projects
None yet
Development

No branches or pull requests

2 participants