[wp-trac] [WordPress Trac] #50105: Remove infinite scrolling behavior from the Media grid

WordPress Trac noreply at wordpress.org
Sun May 2 15:48:25 UTC 2021


#50105: Remove infinite scrolling behavior from the Media grid
-------------------------------------+-------------------------------------
 Reporter:  afercia                  |       Owner:  joedolson
     Type:  defect (bug)             |      Status:  assigned
 Priority:  normal                   |   Milestone:  5.8
Component:  Media                    |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch has-           |     Focuses:  ui, accessibility,
  screenshots needs-testing needs-   |  javascript
  testing-info early                 |
-------------------------------------+-------------------------------------

Comment (by francina):

 Tested

 - MacOS 11.3
 - Safari 14.1
 - WordPress 5.8-alpha-50427-src
 - Twenty Twenty-One 1.3

 After applying it, you have to run`npm run dev` to get the expected
 behavior, might be worth adding it to the testing instructions.

 The patch applies almost cleanly. The total counter doesn't update during
 the upload of the images in grid mode. Only after you refresh the page or
 do another upload. I am going to attach the image.

 That said, I have some feedback:
 - UX/UI - There are no screen options in the grid view, so you can not
 change the number of images loaded in one go
 - Copy - The instruction "Jump to first loaded" is not clear. First what?
 The image of course, but could be clearer. First when? In the last batch.
 What if I want to jump to the first image loaded in the penultimate batch?
 - UX/UI - Have you considered pagination? If you have a large number of
 images, this patch solves the infinite scrolling issues but doesn't
 address the issue of separating the images in smaller batches.

 Thanks!

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/50105#comment:70>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list