[wp-trac] [WordPress Trac] #31793: Theme Switcher: Lazy load theme screenshots

WordPress Trac noreply at wordpress.org
Mon Apr 13 17:11:32 UTC 2015


#31793: Theme Switcher: Lazy load theme screenshots
--------------------------+-------------------------
 Reporter:  ocean90       |       Owner:  ocean90
     Type:  defect (bug)  |      Status:  reopened
 Priority:  high          |   Milestone:  4.2
Component:  Customize     |     Version:  trunk
 Severity:  normal        |  Resolution:
 Keywords:  needs-patch   |     Focuses:  javascript
--------------------------+-------------------------

Comment (by westonruter):

 Replying to [comment:14 ocean90]:
 > [attachment:31793.3.diff] adds a `isRendered` property to
 `api.ThemeControl`. Not sure if it's the right approach.

 This patch makes sense, as we don't want to re-render the content if it
 has already been rendered. However, as with @ryan I still am seeing the
 empty screenshot, but only starting with the 7th theme listed:

 {{{
 <div class="theme-screenshot">
         <img data-src="http://src.wordpress-develop.dev/wp-
 content/themes/twentyten/screenshot.png" alt="">
 </div>
 }}}

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


More information about the wp-trac mailing list