[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 21:25:03 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 Otto42):

 Replying to [comment:18 greenshady]:
 > Will the extra theme screenshots (other than `screenshot.png`) be housed
 in the top level of the theme directory like we do with plugins? Or, will
 there be a `/screenshots` folder, or at least an option to filter it to do
 so?

 Filter wouldn't work. If these shots are on the theme selection screen,
 then core needs to know their location without any code from the theme
 running.

 For the moment, everything is geared towards having core-known-files be in
 the theme root. If we want to add subfolder functionality, then I think a
 better approach would be higher level than just the screenshots. Make the
 theme-file-searcher-thingamabob such that it's capable of looking in named
 subfolders as well as for named files and such.

 For now, I'd stick with a root level naming scheme. screenshot-1.png,
 screenshot-2.png and so forth. This is simple to understand, works fine,
 and using the dash (-) makes it flexible to possibly allow for smarter
 theme-file-searching later (in the same way get_template_part uses the
 dash to separate things).

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


More information about the wp-trac mailing list