[wp-trac] [WordPress Trac] #19816: Theme details should be able to include multiple screenshots

WordPress Trac wp-trac at lists.automattic.com
Wed Feb 22 06:21:14 UTC 2012


#19816: Theme details should be able to include multiple screenshots
----------------------------+------------------
 Reporter:  helenyhou       |       Owner:
     Type:  task (blessed)  |      Status:  new
 Priority:  normal          |   Milestone:  3.4
Component:  Themes          |     Version:  3.3
 Severity:  normal          |  Resolution:
 Keywords:                  |
----------------------------+------------------

Comment (by helenyhou):

 [attachment:19816.diff] is a rough initial pass with heavily commented JS.
 JS used to determine where to pop in the extended details div. Currently
 looks like this: http://cl.ly/3h3Q1Y290N3G083J1L42 - just scraped the
 other screenshots in for demonstration, and the stack would only show for
 themes with multiple screenshots. The stack does intentionally disappear
 when the layer is open. Thinking perhaps the screenshot grid within the
 extended details div should match that of the list table.

 Things to work on:

 * Adding `.has-screenshots` to `.available-theme` divs when applicable
 * Retrieving the multiple screenshots in `get_themes()` and adding to the
 .org API response
 * Appending the images into the extended details div
 * Reflow on window resize
 * Star rating image has a colored (white) background by necessity and
 therefore doesn't match
 * CSS cleanup (colors)

 Tested down to IE8 so far. No JS view remains the same as before.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19816#comment:11>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list