THEME: Foliage - 1.5

[voisula]

Foliage - 1.5

Foliage is really lightweight and simple theme. This theme is optimized by professionals for better search engine visibility, or SEO. Optimization will boost your ranks in search engines and give you more visitors.

Theme URL - ​http://www.themeplate.net/
Author URL -

SVN - ​http://themes.svn.wordpress.org/foliage/1.5
ZIP - ​http://wordpress.org/extend/themes/download/foliage.1.5.zip?nostats=1

Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/foliage/1.4&new_path=/foliage/1.5

All previous tickets for this theme: http://themes.trac.wordpress.org/query?col=id&col=summary&col=keywords&col=owner&col=status&col=resolution&keywords=~theme-foliage&order=id

(Accidentally overwrote this review with the next one... copying and pasting from history.)

Theme Check

PASSED
Previous Issues - FIXED

REQUIRED: Themes must not provide backward compatibility for out-of-date WordPress??? versions, including using function_exists() conditional wrappers for current WordPress??? functions. see:
REQUIRED: Comment reply script (and others) in functions.php should be added in a function hooked into wp_enqueue_scripts.
REQUIRED: Similarly, new widget registration should be hooked into widgets_init
REQUIRED: Sidebars should be registered in a function hooked to widgets_init using an action
REQUIRED: Themes are required to validate and sanitize all untrusted data before entering data into the database, and to escape all untrusted data before being output in the Settings form fields or in the Theme template files. Replace home_url( '/' ) with esc_url( home_url( '/' ) ) in widget_sidebar_search and widget_footer_search
RECOMMENDED: add_custom_background found in the file functions.php. Deprecated since version 3.4. Use add_theme_support( 'custom-background', $args ) instead.
REQUIRED: The theme has a problem with long words in the site title overflowing into the menu area -- this should be contained either with overflow: hidden; or word-wrap: break-word;​​​http://d.pr/i/B2Hf
REQUIRED: Calendars overflow their widget areas in the left and right sidebar areas: ​​http://www.evernote.com/shard/s213/sh/de1bda74-4304-4a61-b372-7aa1d906f771/30ecbd73bf21c538afc50384af090505
REQUIRED: As do search boxes: ​​http://www.evernote.com/shard/s213/sh/acdf68f7-be34-4e68-8eac-e6b86900dd35/f36e0a356c4469ae8f2b2f3c352a9ddd
REQUIRED: "Search for footer" widget does not appear when added to a sidebar
REQUIRED: 404 page should display helpful information besides a 404 not found message -- search form, categories/archives, etc. would be helpful
REQUIRED: Select drop-down menus float oddly in the top sidebar area: ​​http://www.evernote.com/shard/s213/sh/e4e4098e-e744-4b96-8767-7de5d47b9f5c/c1e383c0b40a1aec93b75097a25f666a
REQUIRED: Search results page should display the user's query, as well as useful information (a search form, etc.)
REQUIRED: Large images overflow the sidebars: ​​http://www.evernote.com/shard/s213/sh/88ffcbfe-8ad5-4e2b-a6a4-be48bf003ac7/45bf14018fcbf519a0a68a3cc21e16c2
REQUIRED: Post content can sometimes throw off the layout due to varying height of the image box: ​​http://www.evernote.com/shard/s213/sh/12183d4c-b54c-4723-a3d9-b01a9a7083a1/dce397584c60395dbea21cd25bf4da26
Warnings

REQUIRED: Debug: Use of undefined constant s - assumed 's' on line 9 of /wp-content/themes/foliage/index.php
Previous Issues - NOT FIXED

REQUIRED: Themes are required to use a unique slug as a prefix for anything in the public namespace, including all custom function names, classes, hooks, public/global variables, database entries (Theme options, post custom metadata, etc.) -- my_scripts() function in functions.php should be prefixed with the theme name -- foliage_scripts() or foliage_my_scripts()
Theme Unit Test

REQUIRED: Adding widgets to both left and right sidebars cause the second sidebar to be half the size and drops below the first. ​​http://www.evernote.com/shard/s213/sh/5f434355-6e57-4a38-8f8c-8cedb1547dc5/6f67d71a3852a23ebb94aa6dfad313e8
REQUIRED: Adding widgets to all three footer sidebars causes them to display in a single column instead of floated next to each other.
REQUIRED: Sidebar placement on the 404 page drops the sidebar underneath the content. ​​http://www.evernote.com/shard/s213/sh/cfc8f172-c2af-4918-8e56-88f334bf175f/d8c1e19045ee956f2bcc5c5898b7e8cb
REQUIRED: Custom background color isn't reflected except on the home page, and the home page the implementation is poor. I'd fix or remove support entirely since it's not fully supported.
REQUIRED: Setting a static page as the home page causes the blog page to display without a white background, causing text contrast issues: ​​http://www.evernote.com/shard/s213/sh/0fcfbd28-f9f4-4405-a2e0-db93547e9495/623651d00299b16b187aadba6e8d7eb5
REQUIRED: Improper $content_width value causes video and other media to display outside the available content area. Actual content width is 472px. ​​http://www.evernote.com/shard/s213/sh/2b1b8b51-298e-4693-9352-84236b43ed5c/2b2960dbeea7d9179c34c87ba1abaff1
REQUIRED: Long words overflow the sidebar area: ​​http://www.evernote.com/shard/s213/sh/727d12fb-d8e2-47ca-8745-7bd1fe77c7e1/fc8d6d7052b6621478d60cfbbcc7d772
REQUIRED: Archive pages (Category, Tag, Author, etc.) display the wrong type of post navigation -- instead of Older/Newer?? posts it displays links to specific post titles: ​​http://www.evernote.com/shard/s213/sh/4d419288-6384-4d59-a6e0-1063478a23df/1fb48a4a090f9a25b3de081a9d3116a5
RECOMMENDED: If no sidebars are active, the content area remains narrow and a large empty space exists to the right. It would make for a better design and user experience if the theme's content area adapted to the full width of the wrapper column depending on the number of active sidebars.
RECOMMENDED: Is the footer supposed to be a narrow column? I thought it was full width in the last version, but it's possible I'm not remembering it correctly.
Recommendation

Thanks for submitting your theme! Foliage has several REQUIRED issues noted above that need to be addressed before approval.
Once again, I'll keep this ticket open for 2-3 days -- if you can fix the required issues in that time, please resubmit your theme and comment here with a link to the new ticket and I'll be happy to re-review.
If you have any questions, please feel free to ask in-ticket.

http://themes.trac.wordpress.org/ticket/10116 is the next version of this theme.

I wasn't able to recreate following issue. If it still exists, I would need more info to recreate it to be able to fix it.
REQUIRED: Setting a static page as the home page causes the blog page to display without a white background, causing text contrast issues

Thank you!