[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:  nacin
     Type:  task (blessed)  |      Status:  reopened
 Priority:  normal          |   Milestone:  3.9
Component:  Media           |     Version:  3.5
 Severity:  normal          |  Resolution:
 Keywords:  has-patch       |     Focuses:  javascript
----------------------------+-------------------------

Comment (by kovshenin):

 It looks like the undefined error pops up when scroll() is called for a
 view that has already been removed.

 To reproduce you simply need to change the view while the library is still
 loading additional content, and when it's done it will call scroll() on a
 detached view. The reason @ocean90's steps to reproduce worked (for some
 people) is because "fo" matched some content but "foo" didn't, so there's
 a switch to the UploaderInline view.

 Anyway, this is probably the reason we have that `:visible` check in
 place, so looks like we do need it after all. See
 [attachment:24859.17.diff].

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