[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