[wp-trac] [WordPress Trac] #24859: Media Library does not have a loading indicator
WordPress Trac
noreply at wordpress.org
Wed Mar 5 20:13:07 UTC 2014
#24859: Media Library does not have a loading indicator
-------------------------------------+-------------------------
Reporter: jeffr0 | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.9
Component: Media | Version: 3.5
Severity: normal | Resolution:
Keywords: has-patch needs-refresh | Focuses: javascript
-------------------------------------+-------------------------
Comment (by gcorne):
Spurred by a chat with @kadamwhite, I think that I came up with a simpler
solution as seen in [attachment:24859.3.diff 24859.3.diff].
Instead of binding to a global event bus, we can leverage the existing
{{{updateContent()}} method, which is already bound to the main collection
events. This seems better because the spinner will only show for the
duration of the initial ajax request that populates the the initial set of
attachments. I also incorporated a delay of 600ms before showing the
spinner. The thinking is that showing the spinner anytime there is a query
would leave the user feeling that the browser is slower than it really is.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/24859#comment:21>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list