Make WordPress Core

Opened 3 years ago

Closed 16 months ago

Last modified 15 months ago

#50143 closed enhancement (fixed)

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

Reported by: guddu1315's profile guddu1315 Owned by: joedolson's profile joedolson
Milestone: 5.8 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords:
Focuses: ui, accessibility Cc:

Description

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.

Attachments (2)

50143.png (115.0 KB) - added by afercia 3 years ago.
"Loading" spinner in the media modal
upload.png (928.9 KB) - added by guddu1315 3 years ago.

Download all attachments as: .zip

Change History (12)

#2 @afercia
3 years ago

#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.
Last edited 3 years ago by afercia (previous) (diff)

@afercia
3 years ago

"Loading" spinner in the media modal

#3 @guddu1315
3 years ago

@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.

@guddu1315
3 years ago

#4 @afercia
3 years ago

@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.

#5 @guddu1315
3 years ago

Okay. Understood.
Thanks @afercia

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


3 years ago

#7 @afercia
3 years ago

  • Milestone changed from Awaiting Review to Future Release
  • Version changed from 5.4.1 to 3.5

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

#8 @joedolson
21 months ago

  • Owner set to joedolson
  • Status changed from new to accepted

#9 @joedolson
16 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

Closing this as fixed with the closure of #50105.

#10 @desrosj
15 months ago

  • Milestone changed from Future Release to 5.8
Note: See TracTickets for help on using tickets.