[theme-reviewers] Join the team

Edward Caissie edward.caissie at gmail.com
Mon Apr 18 00:12:43 UTC 2011


@Curtis -

Yes, the wrong textdomain is generally considered a "not-approved"
resolution if for no other reason than it is really just a "find&replace"
function ...

Also note the Author URI; it does not really portend to be one of the a
Theme designer as the minimal content on the site is pointing more towards
and eCommerce blog so it really is getting into the darker shades of gray
related to SEO spamming.

The other items you noted are not too significant, but given the above you
can make a point of noting them in your review comments as the first one is
enough to resolve as "not-approved".


Cais.

On Sun, Apr 17, 2011 at 8:01 PM, Curtis McHale <curtis at curtismchale.ca>wrote:

> 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
> the theme.
> 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.
>
> Curtis McHale
> PH: 604.751.3482
> http://www.curtismchale.ca
> Linkedin: http://www.linkedin.com/in/curtismchale
> Twitter: twitter.com/curtismchale
>
>
>
> 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 ...
>>
>>
>> Cais.
>>
>> 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
>>> http://www.curtismchale.ca
>>> Linkedin: http://www.linkedin.com/in/curtismchale
>>> Twitter: twitter.com/curtismchale
>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110417/b21e684b/attachment.htm>


More information about the theme-reviewers mailing list