[theme-reviewers] Join the team

[SFNdesign]

Thanks for confirming. I'll finish off my notes in a bit and update the ticket. 

Curtis McHale
604.751.3482

On 2011-04-17, at 5:14 PM, Emil Uzelac <emil at themeid.com> wrote:

> 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
> 
> 
> _______________________________________________
> 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/6e97eff4/attachment-0001.htm>