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

WordPress Trac noreply at wordpress.org
Fri Mar 7 22:54:12 UTC 2014


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

Comment (by kadamwhite):

 Added one option for triggering the parent AttachmentBrowser's toolbar
 spinner from within the Attachments view, using `.views.parent.toolbar`.
 Reaching up the sub-view chain feels a little janky. These views share a
 collection so I'd have preferred to do this with collection events, but
 since we DIY'd all the AJAX (collection.more(), etc) I'm finding that none
 of the usual request/sync events fire on `this.collection`.

 .11.diff I think is ready for prime-time; on this one
 ([attachment:24859.12.diff]) I'd like some peer review.

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


More information about the wp-trac mailing list