[theme-reviewers] Join the team

[Emil Uzelac]

sorry @Cais, I sent at the same time :(
*----*
*Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:
emil at themeid.com | http://themeid.com
Make everything as simple as possible, but not simpler. - Albert Einstein



On Sun, Apr 17, 2011 at 7:12 PM, Edward Caissie <edward.caissie at gmail.com>wrote:

> @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
>>
>>
>
> _______________________________________________
> 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/b05d6d4d/attachment.htm>