[wp-trac] [WordPress Trac] #19816: Theme details should be able to include multiple screenshots
WordPress Trac
wp-trac at lists.automattic.com
Sat Feb 25 20:38:25 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.2.diff] adds in `.has-screenshots` (needs
[attachment:stack.png] in `wp-admin/images`, not added in the patch) and
handles screenshots as an array, as well as appending them in to the
extended details div.
Thinking further, we could probably just have both `get_themes()` and the
.org API response continue to return the first screenshot, as it does now,
and then just the number of screenshots. The full URL for all of the
screenshots isn't needed, since naming and file extension (.png) should be
consistent for additional images. The JS in this patch only receives the
total number of images and builds the image src URL from there.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/19816#comment:12>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list