[wp-trac] [WordPress Trac] #24859: Media Library does not have a loading indicator

[WordPress Trac]

#24859: Media Library does not have a loading indicator
------------------------------+-----------------------------
 Reporter:  jeffr0            |       Owner:  wonderboymusic
     Type:  task (blessed)    |      Status:  reopened
 Priority:  normal            |   Milestone:  3.9
Component:  Media             |     Version:  3.5
 Severity:  normal            |  Resolution:
 Keywords:  has-patch commit  |     Focuses:  javascript
------------------------------+-----------------------------

Comment (by gcorne):

 [attachment:24859.13.diff 24859.13.diff] builds on @kadamwhite's work in
 [attachment:24859.12.diff 24859.12.diff] by fixing the race condition that
 @nacin reported and making the delay value a property.

 >300 felt pretty good to me, but chatting with nacin he wondered if
 perhaps we should leave the timeout at 0 and debounce the hiding instead,
 as if you type fast you get jittering of the spinner restarting.

 I would rather not show the spinner in cases where the async request is
 fast because when a spinner is shown the user feels like the UI
 responsiveness is slower than it actually is. There is a
 [http://www.yuiblog.com/blog/2010/11/11/improving-perceived-site-
 performance-with-spinners/ YUI post] about this that links to a now
 defunct jsfiddle, so trusting the "results" requires blind faith. :)

 Google Chrome also is doing some interesting
 [http://www.yuiblog.com/blog/2010/11/11/improving-perceived-site-
 performance-with-spinners/ stuff] in this area.

 With the race condition fixed, I am not sure if there is anything we need
 to do in terms of controlling when the spinner is hidden. @helen, do you
 agree?

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