[wp-trac] [WordPress Trac] #19816: Theme details should be able to include multiple screenshots
WordPress Trac
wp-trac at lists.automattic.com
Wed Feb 29 05:57:48 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.4.diff] is a refresh to use the beautiful `WP_Theme` for
the screenshot count. Theme install side will likely need to be tweaked
later to accommodate whatever gets returned from the .org API, and then
should probably have variable names actually match. Would like some eyes
on the JS and then to get this in for a look at UX. It all works nicely -
add some appropriately named screenshots to an installed theme folder for
a look. And [attachment:stack.png].
One note about the UX that started with [attachment:19816.3.diff] - the
details div snaps shut when the window is resized. We can look at other
behaviors, but this seemed the most sane for a first go. Nothing
particularly breaks on window resize, but there is some funkiness without
the function on `$(window).resize()`.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/19816#comment:15>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list