[theme-reviewers] Join the team
curtis at curtismchale.ca
Mon Apr 18 00:01:59 UTC 2011
I have a few questions about my first theme review.
1. The theme uses the text domain of Twenty Ten in a few spots. If I
remember correctly from lurking on the list this is a reason for failure of
2. They use a widget for the menu and instruct the user to add the Pages
widget. It seems to me that it's a much better option to use a WordPress
menu which defaults to list pages. I don't believe this is an issue for a
fail but probably a good suggestion as an update to the user.
3. The CSS fails on the background-color since it's value is set to 'none'
when it should be set to 'transparent'. Would this typically be a reason for
failure if all other CSS is okay?
4. There are a number of other 'recommended' items in Theme Check but these
are not a reason to fail a theme correct?
I could probably go further in to the theme but I wanted to double check on
the text domain item before I did.
On Sat, Apr 16, 2011 at 4:39 PM, Edward Caissie <edward.caissie at gmail.com>wrote:
> Here ya go: http://themes.trac.wordpress.org/ticket/3320
> This will get you started ...
> On Sat, Apr 16, 2011 at 7:15 PM, Curtis McHale <curtis at curtismchale.ca>wrote:
>> Figured I'd finally get off my butt and join the team after looking at the
>> list for a few months. My trac name is curtismchale
>> Curtis McHale
>> PH: 604.751.3482
>> Linkedin: http://www.linkedin.com/in/curtismchale
>> Twitter: twitter.com/curtismchale
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the theme-reviewers