Load spinner missing from admin user list

Contribute Bug

AI-generated summary

The discussion Load spinner missing from admin user list revolves around the absence of a loading spinner in the admin user list. trusktr initially reported the issue, providing screen recordings to demonstrate the problem. However, merefield suggested that the spinner is actually present, but the delay is too short to be noticeable. trusktr throttled the network to 3G, but still couldn’t see the spinner. merefield pointed out that the spinner is indeed in the code and attributed its absence to fast data retrieval. david explained that the spinner is appended at the bottom of the list, which is often scrolled off the screen, and proposed a fix. merefield agreed with david, noting that the list is truncated immediately, allowing the spinner to be visible. A pull request (#5) aims to improve the situation.

1

Admin pages are no longer showing load animations.

I just updated a site that was several years old. On the old site, loading new data, for example in the users list, would show a loading spinner. But now there is no spinner (which could be more misleading on slower networks). Here’s what I see:

You can see that as I switch tabs, there’s no loading spinner. I was looking at the old site earlier today and it had a spinner.

If I full refresh the page, then the spinner will appear:

So it’s missing only on dynamic content swap.

2 Likes
2

I don’t believe this is correct. It’s just that the delay is so very short you never see the spinner.

If you keep switching you can sometimes glimpse it.

3

Looks like still no loader even if I throttle the network to 3G:

2 Likes
4

Here is the spinner in the code:

I think it’s just the speed of the data retrieval that is so fast (and almost certainly cached) it doesn’t appear.

2 Likes
5

Yeah there is a spinner, but it’s just appended at the bottom of the list, so it’s normally scrolled off the bottom of the screen. This should improve the situation:

https://github.com/discourse/discourse/pull/31856

4 Likes
6

Aah, better that as the list is first truncated this is reflected in the UI immediately so you can actually see the spinner …

2 Likes
7

This topic was automatically closed after 6 days. New replies are no longer allowed.