Add a loader in the media library while next set of images are loaded

[guddu1315]

Hello

In the media library, only a few numbers of images are loaded at first. The next set of images are loaded using Infinite scrolling. This works fine on sites that are fast.

But when we have a slow site. Maybe because of the server or a plugin or a paid theme or may be due to lots of heavy images.

It takes too much time to load the next set of images, and there is no way for the user to know if the next set of images are still being loaded or its the end of the media items.

So the following things would be a great addition.

1) Adding a loader while the next set of images are fetched.
2) When all images are loaded. Displaying a message like "End of media" or any other suitable message.

Thank you.

[SergeyBiryukov]

Related: #50105

[afercia]

#50105 proposes to remove the infinite scrolling behavior, which would drastically solve this issue.

Worth noting the media grid does have a "loader" icon: the standard spinner appears in the top toolbar.

However:

• In the media modal the toolbar has a fixed position, thus it's always visible (see screenshot below)
• On the Media Library page, the toolbar scrolls together with the page so unless users have a very tall display, the toolbar is out of view after they scrolled the page. Of course, this makes the spinner icon pointless in the Media Library page.

[afercia]

"Loading" spinner in the media modal

[guddu1315]

@afercia true for media library popup.
I was talking about upload.php like you said the toolbar is not sticky so when users scroll down they can't see that spinner. So that spinner has to be in the bottom with a load more button and a message indicating its end of images.

[afercia]

@guddu1315 thanks for the clarification. Yeah, I got your point but maybe my previous comment wasn't clear :)
Please consider this ticket may become no longer relevant if the change proposed on #50105 will be merged. We will see how it goes.

[guddu1315]

Okay. Understood.
Thanks @afercia

[afercia]

This ticket was discussed during today's accessibility bug-scrub: agreed to move to Future Release for now, pending #50105.

[joedolson]

Closing this as fixed with the closure of #50105.