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

WordPress Trac noreply at wordpress.org
Sun May 2 16:29:22 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 joedolson):

 Thanks, @francina!

 In a media meeting, we discussed that making the number of images a
 setting would probably be a future release issue, as there are a number of
 UI questions to resolve for that. That's also true for pagination, as
 implementing pagination into the media library would be a much more
 substantial change to the way it works.

 The total counter updating is still an unresolved issue; it's the major
 outstanding item to figure out.

 The language for the 'jump to' is a hard problem. It can't be 'image',
 since the media library contains attachments of all types. It could be
 'attachment', I suppose; I'm not sure how readily understood that would
 be. But there's a question of how verbose it makes sense to be, e.g. "Jump
 to first attachment loaded in this request" seems awfully verbose.

 I have no particular bias about the specific language; happy to hear
 suggestions! The aim is certainly to reach something that will be easily
 understood, but preferably also concise.

 Can you tell me what in the patch didn't apply cleanly for you? This patch
 should be clean; it was fresh yesterday, so I'm not sure why it wouldn't
 apply cleanly.

 I'm not sure what `npm run dev` does; it's not a command I've ever used.

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


More information about the wp-trac mailing list