#6349 closed theme (live)
THEME: Grisaille - 1.2
| Reported by: |
nudge
|
Owned by: |
sixhours
|
|---|---|---|---|
| Priority: | Keywords: | theme-grisaille | |
| Cc: | nudgeme@… |
Description
Grisaille - 1.2
A classic and simple two-column design adjusted for mobile browsing. Latest post will display larger image when inserted with post thumbnail/featured image option. Twitter and Facebook links available via the theme's option page, as well as customizable menu, header image and background. It's pronounced "griz-eye".
Theme URL - http://nudgedesign.ca/wordpress-themes/grisaille/
Author URL - http://www.nudgedesign.ca
SVN - http://themes.svn.wordpress.org/grisaille/1.2
ZIP - http://wordpress.org/extend/themes/download/grisaille.1.2.zip?nostats=1
Diff with previous version: http://themes.trac.wordpress.org/changeset?old_path=/grisaille/1.1&new_path=/grisaille/1.2
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-grisaille&order=id
@
Thank you! I'm marking as Approved, with one fix to be applied in the next release.
Pages should not display a "No comments" message--unless comments are enabled, no comment info should be displayed. - Not fixed. Still displaying "0 comments" and "Sorry, comments are closed" message on Pages.
Issues from the previous review:
Theme functions in functions.php should have a prefix specific to the theme, a la "grisaille_admin_header_style" FIXED
Long title, tagline, or menu have the potential to throw off the layout (see attached screencap longtitle.jpg) FIXED
"Notify me" areas after comment form are floating oddly (see attached screencap notify.jpg) FIXED
Wide content (images with captions) overflows the content area (see attached screencap widecontentoverflow.jpg) FIXED
wp_head() should come directly before closing <head> tag in header.php FIXED