[wp-trac] [WordPress Trac] #26968: Theme Administration Screen: Not enough themes displayed at high resolutions

[WordPress Trac]

#26968: Theme Administration Screen: Not enough themes displayed at high
resolutions
----------------------------+------------------------------
 Reporter:  Viper007Bond    |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  low             |   Milestone:  Awaiting Review
Component:  Administration  |     Version:  3.8
 Severity:  minor           |  Resolution:
 Keywords:  needs-patch     |     Focuses:  ui
----------------------------+------------------------------

Comment (by nacin):

 This is funny. matveb set it up to load 20 themes at a time. This is for
 performance. It has all the data, but when you have a ton of themes and it
 has to download all of those screenshots. The issue here is that number
 should probably be high enough to cover even an insane viewport.

 So, when I added a fallback for when there's a JS syntax error (or no JS
 at all, of course). But I didn't limit it to 20 at a time, because there's
 no pagination fallback. It just prints all of them. If/when the JS
 successfully loads, it empties the page and re-renders the views. That's
 the flicker you're seeing. No one else has noticed it because it'd happen
 out of the viewport.

 I have some familiarity with how browsers load images, but I'm not sure if
 removing a not-yet-loaded image from the DOM would stop it from being
 downloaded (thus defeating the purpose of the 20-at-a-time).

 I can't answer the question of why "add new theme" shows up. Probably
 because everyone would always scroll down and they'd never see it until
 they reached the end of their list.

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