[wp-trac] [WordPress Trac] #33311: TB_window spinner should probably disappear when loading is done

[WordPress Trac]

#33311: TB_window spinner should probably disappear when loading is done
-------------------------+-------------------------------------------------
 Reporter:  niklasbr     |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Plugins      |     Version:  4.2.4
 Severity:  normal       |  Resolution:
 Keywords:  has-patch    |     Focuses:  ui, javascript, administration,
  dev-feedback           |  performance
-------------------------+-------------------------------------------------

Comment (by afercia):

 Screenshot to illustrate the current situation, see how using Chrome the
 entire TB_window area is continuously repainted even when the spinner is
 not visible because is behind the iframe. Same happens in Firefox, see the
 video attached above. We should try to avoid to use an animated GIF as
 background image in a so large area. [attachment:33311.test.patch] creates
 a small 20px x 20px area and sets the background GIF there.
 Thinking we should combine the two patches and use the best of the two
 approaches:
 - when using an animated GIF as background image, always set it on the
 smallest possible area
 - always remove the background image when it's not needed anymore

 [[Image(https://cldup.com/CUWrn89qCF.png)]]

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