THEME: Grisaille - 1.3.1

[nudge]

Grisaille - 1.3.1

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.3.1
ZIP - ​http://wordpress.org/extend/themes/download/grisaille.1.3.1.zip?nostats=1

Diff with previous version: ​http://themes.trac.wordpress.org/changeset?old_path=/grisaille/1.3&new_path=/grisaille/1.3.1

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

[life.object]

Review Summary:

• Diff-review only.
• Previous version of Theme not-approved.
• Required issues from previous ticket.

Theme Plugin Test:

Plugin "Theme Check": Pass
Plugin "Debogger": Pass
Plugin "Log Deprecated Notices": None

Issues from Previous Ticket:

Fixed: Pages should not display a "No comments" message.

Theme Unit Test:

Recommended Issues:

• Long strings in menu drop down. e.g(aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa), You can fix it by using "word-wrap: break-word;".

Review:

• Required issues have been fixed for the previously not-approved Theme.
• Current version looks good and without apparent issues.

Theme Recommendation:

• Approved
• Ticket being resolved as approved. Please address recommended issues in the next Theme revision.

[nudge]

Hi, wondering when this will be put live.

Thanks!

[cais]

It has brought to our attention the use of "Nick's Fonts" may not be compatible with the current WordPress guidelines and compatible font licenses.

Please see: ​http://codex.wordpress.org/Theme_Review#GPL-Compatible_Font_Licenses

This issue should be addressed as soon as possible. Thanks.

[nudge]

Hi,

Thanks for bringing this to my attention.

I'm a little confused though. I picked up this font from Font Squirrel with an @font-face kit.
​http://www.fontsquirrel.com/fonts/WoodenNickelBlack

Can you clarify where I misunderstood the licenses if I can't use this one?

Thanks!

[cais]

@nudge - The license you included appears to be more what the font author's intent is but not a specifically stated license as noted on the link I provided.

Fonts bundled with Themes submitted to the WordPress Theme directory are required to be licensed under one of the following font licenses ...

The font you bundled does not explicitly state being licensed under one of the licenses listed that are considered compatible.

Perhaps you could contact the font author and ask if they would be willing to choose a GPL-compatible license to be used with the fonts.

[nudge]

Thanks for clarifying, Cais.

I'll get in touch with the author and see what he says.

Thanks!

Sonia