<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">thanks!<br clear="all"></font></font></font><div><strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div class="gmail_quote">On Wed, Apr 20, 2011 at 12:09 AM, Shawn Grundy <span dir="ltr"><<a href="mailto:smgrundy@live.com">smgrundy@live.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div>
done with both...<div><br></div><div>Shawn<br><br><hr>From: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a><br>Date: Tue, 19 Apr 2011 22:02:54 -0500<div><div></div><div class="h5"><br>To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
Subject: Re: [theme-reviewers] theme-reviewers Digest, Vol 11, Issue 92<br><br><font color="#333333"><font size="2"><div><font color="#333333"><font size="2">Two tickets assigned to: </font></font><span style="font-family:arial, helvetica, sans-serif"><span style="font-family:arial;color:rgb(0, 0, 0)">smgrundy</span></span></div>
<div><span style="font-family:arial, helvetica, sans-serif"><span style="font-family:arial;color:rgb(0, 0, 0)"><br></span></span></div>
<font face="arial,helvetica,sans-serif"><a href="http://themes.trac.wordpress.org/ticket/3499" target="_blank">http://themes.trac.wordpress.org/ticket/3499</a></font></font></font><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><a href="http://themes.trac.wordpress.org/ticket/3499" target="_blank"></a><a href="http://themes.trac.wordpress.org/ticket/3500" target="_blank">http://themes.trac.wordpress.org/ticket/3500</a></font></font></font></div>
<div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif"><a href="http://themes.trac.wordpress.org/ticket/3500" target="_blank"></a>Thanks,</font></font></font></div>
<div><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Emil<br clear="all"></font></font></font><div><strong><font face="arial, helvetica, sans-serif">----</font></strong></div><div><font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div>On Tue, Apr 19, 2011 at 9:56 PM, Shawn Grundy <span dir="ltr"><<a href="mailto:smgrundy@live.com" target="_blank">smgrundy@live.com</a>></span> wrote:<br><blockquote style="border-left:1px #ccc solid;padding-left:1ex">
<div>
I will take a theme or two Emil if you have them!<div><br></div><div>username: smgrundy<br><br><hr>From: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a><br>Date: Tue, 19 Apr 2011 18:27:59 -0500<div>
<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br></div>Subject: Re: [theme-reviewers] theme-reviewers Digest, Vol 11, Issue 92<div><div></div><div>
<br><br><font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">No worries and thank you!<br clear="all"></font></font></font><div><strong><font face="arial, helvetica, sans-serif">----</font></strong></div>
<div>
<font face="arial, helvetica, sans-serif"><strong>Emil Uzelac</strong> | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com/" target="_blank">http://themeid.com</a></font></div>
<div><font color="#999999" face="arial, helvetica, sans-serif">Make everything as simple as possible, but not simpler. - Albert Einstein</font></div><br>
<br><br><div>On Tue, Apr 19, 2011 at 6:27 PM, Tammie Lister <span dir="ltr"><<a href="mailto:karmatosed@gmail.com" target="_blank">karmatosed@gmail.com</a>></span> wrote:<br><blockquote style="border-left:1px #ccc solid;padding-left:1ex">
Thanks will review that one tomorrow.<br>
<br>
Regards<br>
Tammie Lister<br>
<br>
On 20 Apr 2011, at 00:13, <a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a> wrote:<br>
<br>
> Send theme-reviewers mailing list submissions to<br>
> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
><br>
> To subscribe or unsubscribe via the World Wide Web, visit<br>
> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
> or, via email, send a message with subject or body 'help' to<br>
> <a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a><br>
><br>
> You can reach the person managing the list at<br>
> <a href="mailto:theme-reviewers-owner@lists.wordpress.org" target="_blank">theme-reviewers-owner@lists.wordpress.org</a><br>
><br>
> When replying, please edit your Subject line so it is more specific<br>
> than "Re: Contents of theme-reviewers digest..."<br>
><br>
><br>
> Today's Topics:<br>
><br>
> 1. Re: Theme review completed (Emil Uzelac)<br>
><br>
><br>
> ----------------------------------------------------------------------<br>
><br>
> Message: 1<br>
> Date: Tue, 19 Apr 2011 18:13:12 -0500<br>
> From: Emil Uzelac <<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>><br>
> Subject: Re: [theme-reviewers] Theme review completed<br>
> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
> Message-ID: <BANLkTimk8CPe8txXC5OG-Fm=<a href="mailto:3y7jwfPR5A@mail.gmail.com" target="_blank">3y7jwfPR5A@mail.gmail.com</a>><br>
> Content-Type: text/plain; charset="iso-8859-1"<br>
><br>
> Sure <a href="http://themes.trac.wordpress.org/ticket/3498" target="_blank">http://themes.trac.wordpress.org/ticket/3498</a><br>
> *----*<br>
> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:<br>
> <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com" target="_blank">http://themeid.com</a><br>
> Make everything as simple as possible, but not simpler. - Albert Einstein<br>
><br>
><br>
><br>
> On Tue, Apr 19, 2011 at 6:11 PM, Tammie Lister <<a href="mailto:tlister@logicalbinary.com" target="_blank">tlister@logicalbinary.com</a>>wrote:<br>
><br>
>> I've completed another theme review tonight and would like to review<br>
>> another please tomorrow taking the feedback on board of extra things I<br>
>> should be looking for. Again happy either BuddyPress or WordPress.<br>
>><br>
>> <a href="http://themes.trac.wordpress.org/ticket/3487" target="_blank">http://themes.trac.wordpress.org/ticket/3487</a> was one did tonight.<br>
>><br>
>> Regards,<br>
>> Tammie Lister<br>
>><br>
>> On Tuesday, 19 April 2011 at 23:58,<br>
>> <a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a> wrote:<br>
>><br>
>> Send theme-reviewers mailing list submissions to<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>><br>
>> To subscribe or unsubscribe via the World Wide Web, visit<br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>> or, via email, send a message with subject or body 'help' to<br>
>> <a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a><br>
>><br>
>> You can reach the person managing the list at<br>
>> <a href="mailto:theme-reviewers-owner@lists.wordpress.org" target="_blank">theme-reviewers-owner@lists.wordpress.org</a><br>
>><br>
>> When replying, please edit your Subject line so it is more specific<br>
>> than "Re: Contents of theme-reviewers digest..."<br>
>><br>
>><br>
>> Today's Topics:<br>
>><br>
>> 1. Re: @smgrundy [WordPress Themes] #3563: THEME: For<br>
>> Women-Female - 1.1 (Shawn Grundy)<br>
>> 2. Re: @smgrundy [WordPress Themes] #3563: THEME: For<br>
>> Women-Female - 1.1 (Emil Uzelac)<br>
>> 3. After 3367 - request (rex posadas)<br>
>> 4. Re: Another ticket (Curtis McHale)<br>
>> 5. Re: Another ticket (Chip Bennett)<br>
>><br>
>><br>
>> ----------------------------------------------------------------------<br>
>><br>
>> Message: 1<br>
>> Date: Tue, 19 Apr 2011 14:21:38 -0700<br>
>> From: Shawn Grundy <<a href="mailto:smgrundy@live.com" target="_blank">smgrundy@live.com</a>><br>
>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563:<br>
>> THEME: For Women-Female - 1.1<br>
>> To: <<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>><br>
>> Message-ID: <COL102-W23982B423D7C0CA1596795AE900@phx.gbl><br>
>> Content-Type: text/plain; charset="iso-8859-1"<br>
>><br>
>><br>
>> done<br>
>><br>
>> From: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a><br>
>> Date: Tue, 19 Apr 2011 15:50:22 -0500<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563: THEME:<br>
>> For Women-Female - 1.1<br>
>><br>
>> ----Emil Uzelac | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:<br>
>> <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com" target="_blank">http://themeid.com</a><br>
>><br>
>><br>
>> Make everything as simple as possible, but not simpler. - Albert Einstein<br>
>><br>
>><br>
>><br>
>> On Tue, Apr 19, 2011 at 3:49 PM, WordPress Themes <<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>> wrote:<br>
>><br>
>><br>
>> #3563: THEME: For Women-Female - 1.1<br>
>><br>
>> ------------------------+-----------------------------------<br>
>><br>
>> Reporter: Stilisten | Owner: smgrundy<br>
>><br>
>> Type: theme | Status: assigned<br>
>><br>
>> Resolution: | Keywords: theme-for-women-female<br>
>><br>
>> ------------------------+-----------------------------------<br>
>><br>
>> Changes (by emiluzelac):<br>
>><br>
>><br>
>><br>
>> * owner: => smgrundy<br>
>><br>
>> * status: new => assigned<br>
>><br>
>><br>
>><br>
>><br>
>><br>
>> Comment:<br>
>><br>
>><br>
>><br>
>> @smgrundy since you reviewed first time here it is once again. This should<br>
>><br>
>> be good to go as the author fixed all of the items.<br>
>><br>
>><br>
>><br>
>> --<br>
>><br>
>> Ticket URL: <<a href="http://themes.trac.wordpress.org/ticket/3563#comment:1" target="_blank">http://themes.trac.wordpress.org/ticket/3563#comment:1</a>><br>
>><br>
>> WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>
>><br>
>> WordPress.org Theme Directory Reviews<br>
>><br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>> -------------- next part --------------<br>
>> An HTML attachment was scrubbed...<br>
>> URL: <<br>
>> <a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e2e4a631/attachment-0001.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e2e4a631/attachment-0001.htm</a><br>
>>><br>
>><br>
>> ------------------------------<br>
>><br>
>> Message: 2<br>
>> Date: Tue, 19 Apr 2011 16:29:11 -0500<br>
>> From: Emil Uzelac <<a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a>><br>
>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563:<br>
>> THEME: For Women-Female - 1.1<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Message-ID: <<a href="mailto:BANLkTikKrPxMtJrueT%2BCK-pt4BfWN1nJpA@mail.gmail.com" target="_blank">BANLkTikKrPxMtJrueT+CK-pt4BfWN1nJpA@mail.gmail.com</a>><br>
>> Content-Type: text/plain; charset="iso-8859-1"<br>
>><br>
>> Thank you!<br>
>> *----*<br>
>> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:<br>
>> <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com" target="_blank">http://themeid.com</a><br>
>> Make everything as simple as possible, but not simpler. - Albert Einstein<br>
>><br>
>><br>
>><br>
>> On Tue, Apr 19, 2011 at 4:21 PM, Shawn Grundy <<a href="mailto:smgrundy@live.com" target="_blank">smgrundy@live.com</a>> wrote:<br>
>><br>
>> done<br>
>><br>
>> ------------------------------<br>
>> From: <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a><br>
>> Date: Tue, 19 Apr 2011 15:50:22 -0500<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563: THEME:<br>
>> For Women-Female - 1.1<br>
>><br>
>><br>
>><br>
>> *----*<br>
>> *Emil Uzelac* | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E:<br>
>> <a href="mailto:emil@themeid.com" target="_blank">emil@themeid.com</a> | <a href="http://themeid.com" target="_blank">http://themeid.com</a><br>
>> Make everything as simple as possible, but not simpler. - Albert Einstein<br>
>><br>
>><br>
>><br>
>> On Tue, Apr 19, 2011 at 3:49 PM, WordPress Themes <<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>> wrote:<br>
>><br>
>> #3563: THEME: For Women-Female - 1.1<br>
>> ------------------------+-----------------------------------<br>
>> Reporter: Stilisten | Owner: smgrundy<br>
>> Type: theme | Status: assigned<br>
>> Resolution: | Keywords: theme-for-women-female<br>
>> ------------------------+-----------------------------------<br>
>> Changes (by emiluzelac):<br>
>><br>
>> * owner: => smgrundy<br>
>> * status: new => assigned<br>
>><br>
>><br>
>> Comment:<br>
>><br>
>> @smgrundy since you reviewed first time here it is once again. This should<br>
>> be good to go as the author fixed all of the items.<br>
>><br>
>> --<br>
>> Ticket URL: <<a href="http://themes.trac.wordpress.org/ticket/3563#comment:1" target="_blank">http://themes.trac.wordpress.org/ticket/3563#comment:1</a>><br>
>> WordPress Themes <<a href="http://themes.trac.wordpress.org/" target="_blank">http://themes.trac.wordpress.org/</a>><br>
>> WordPress.org Theme Directory Reviews<br>
>><br>
>><br>
>><br>
>> _______________________________________________ theme-reviewers mailing<br>
>> list <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>><br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>><br>
>> -------------- next part --------------<br>
>> An HTML attachment was scrubbed...<br>
>> URL: <<br>
>> <a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/5999840f/attachment-0001.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/5999840f/attachment-0001.htm</a><br>
>>><br>
>><br>
>> ------------------------------<br>
>><br>
>> Message: 3<br>
>> Date: Tue, 19 Apr 2011 15:33:07 -0700 (PDT)<br>
>> From: rex posadas <<a href="mailto:rexposadas@yahoo.com" target="_blank">rexposadas@yahoo.com</a>><br>
>> Subject: [theme-reviewers] After 3367 - request<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Message-ID: <<a href="mailto:969894.75830.qm@web36504.mail.mud.yahoo.com" target="_blank">969894.75830.qm@web36504.mail.mud.yahoo.com</a>><br>
>> Content-Type: text/plain; charset="us-ascii"<br>
>><br>
>> Hello,<br>
>><br>
>> I just reviewed 3367, I am ready for another theme.<br>
>><br>
>> username: rxn<br>
>><br>
>> Thank You.<br>
>><br>
>> Rex<br>
>> -------------- next part --------------<br>
>> An HTML attachment was scrubbed...<br>
>> URL: <<br>
>> <a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/96396658/attachment-0001.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/96396658/attachment-0001.htm</a><br>
>>><br>
>><br>
>> ------------------------------<br>
>><br>
>> Message: 4<br>
>> Date: Tue, 19 Apr 2011 15:44:52 -0700<br>
>> From: Curtis McHale <<a href="mailto:curtis@curtismchale.ca" target="_blank">curtis@curtismchale.ca</a>><br>
>> Subject: Re: [theme-reviewers] Another ticket<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Message-ID: <BANLkTi==<a href="mailto:o%2BEbjW9CVjQ3BOcsmrZ4MAVXaQ@mail.gmail.com" target="_blank">o+EbjW9CVjQ3BOcsmrZ4MAVXaQ@mail.gmail.com</a>><br>
>> Content-Type: text/plain; charset="iso-8859-1"<br>
>><br>
>> The theme I'm reviewing has JS that detects the browser version and tells<br>
>> them to update (<a href="http://browser-update.org/" target="_blank">http://browser-update.org/</a>). There is not a theme option<br>
>> to<br>
>> remove this function or information in the description.<br>
>><br>
>> What is the theme review stance on this feature?<br>
>><br>
>> There are also some list styles missing in comments. I don't think this is<br>
>> a<br>
>> reason for failure but should be noted and requested for update on the next<br>
>> version. Is that correct?<br>
>><br>
>> The theme options are strictly a page that suggests the user donate if they<br>
>> like the theme. I know that credit links are okay but I'm not sure about<br>
>> this instance.<br>
>><br>
>> The theme calls loop-archive.php through get_template_part but the file<br>
>> doesn't actually exist. Fortunately it defaults to loop.php but I'm<br>
>> wondering how this would be addressed. The theme works but I would think<br>
>> this should be corrected in the next version either by just using loop.php<br>
>> or by adding the file that is being called. Is that correct?<br>
>><br>
>> The theme doesn't support post_formats. I thought that they were supposed<br>
>> to<br>
>> at least declare it and support whichever they picked. I don't see this<br>
>> information on the Theme Review page so is this assumption incorrect?<br>
>><br>
>> Curtis McHale<br>
>> PH: 604.751.3482<br>
>> <a href="http://www.curtismchale.ca" target="_blank">http://www.curtismchale.ca</a><br>
>> Linkedin: <a href="http://www.linkedin.com/in/curtismchale" target="_blank">http://www.linkedin.com/in/curtismchale</a><br>
>> Twitter: <a href="http://twitter.com/curtismchale" target="_blank">twitter.com/curtismchale</a><br>
>> -------------- next part --------------<br>
>> An HTML attachment was scrubbed...<br>
>> URL: <<br>
>> <a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/9b218d3d/attachment-0001.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/9b218d3d/attachment-0001.htm</a><br>
>>><br>
>><br>
>> ------------------------------<br>
>><br>
>> Message: 5<br>
>> Date: Tue, 19 Apr 2011 17:58:21 -0500<br>
>> From: Chip Bennett <<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>><br>
>> Subject: Re: [theme-reviewers] Another ticket<br>
>> To: <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> Message-ID: <BANLkTi=<a href="mailto:Z_Qb3E9msQQNLhMAr70_Hrk5CmA@mail.gmail.com" target="_blank">Z_Qb3E9msQQNLhMAr70_Hrk5CmA@mail.gmail.com</a>><br>
>> Content-Type: text/plain; charset="iso-8859-1"<br>
>><br>
>> Comments inline...<br>
>><br>
>> On Tue, Apr 19, 2011 at 5:44 PM, Curtis McHale <<a href="mailto:curtis@curtismchale.ca" target="_blank">curtis@curtismchale.ca</a><br>
>>> wrote:<br>
>><br>
>> The theme I'm reviewing has JS that detects the browser version and tells<br>
>> them to update (<a href="http://browser-update.org/" target="_blank">http://browser-update.org/</a>). There is not a theme option<br>
>> to remove this function or information in the description.<br>
>><br>
>> What is the theme review stance on this feature?<br>
>><br>
>><br>
>> Incredibly annoying, but nothing prohibits it. I would *require* a mention<br>
>> of the "feature" in Theme documentation (readme.txt).<br>
>><br>
>><br>
>> There are also some list styles missing in comments. I don't think this is<br>
>> a reason for failure but should be noted and requested for update on the<br>
>> next version. Is that correct?<br>
>><br>
>><br>
>> It depends on how egregious the style omission is. If lists within comments<br>
>> are inheriting the comment-list styling, I would fail for that. If they<br>
>> merely don't have proper indenting/bullets, I would perhaps say *required*<br>
>> in next revision.<br>
>><br>
>><br>
>> The theme options are strictly a page that suggests the user donate if they<br>
>> like the theme. I know that credit links are okay but I'm not sure about<br>
>> this instance.<br>
>><br>
>><br>
>> Also nothing prohibiting this. But I would balk at calling such a page an<br>
>> "Options" page. You could perhaps require the page to be titled "Theme<br>
>> Documentation" (etc.) rather than "Theme Options".<br>
>><br>
>><br>
>> The theme calls loop-archive.php through get_template_part but the file<br>
>> doesn't actually exist. Fortunately it defaults to loop.php but I'm<br>
>> wondering how this would be addressed. The theme works but I would think<br>
>> this should be corrected in the next version either by just using loop.php<br>
>> or by adding the file that is being called. Is that correct?<br>
>><br>
>><br>
>> That could be for Child-Theme support (allowing e.g. a Child Theme to<br>
>> include "loop-archive.php", and have it included). Due to the inherent<br>
>> callback nature of the function, this is a correct implementation.<br>
>><br>
>><br>
>> The theme doesn't support post_formats. I thought that they were supposed<br>
>> to at least declare it and support whichever they picked. I don't see this<br>
>> information on the Theme Review page so is this assumption incorrect?<br>
>><br>
>><br>
>> Post Formats are not required to be implemented. However, if the Theme has<br>
>> custom loops for any of the Post Format types, then it is *required* to<br>
>> support the core feature.<br>
>><br>
>> Chip<br>
>> -------------- next part --------------<br>
>> An HTML attachment was scrubbed...<br>
>> URL: <<br>
>> <a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/01dbdb66/attachment.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/01dbdb66/attachment.htm</a><br>
>>><br>
>><br>
>> ------------------------------<br>
>><br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>><br>
>><br>
>> End of theme-reviewers Digest, Vol 11, Issue 90<br>
>> ***********************************************<br>
>><br>
>><br>
>><br>
>> _______________________________________________<br>
>> theme-reviewers mailing list<br>
>> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
>> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
>><br>
>><br>
> -------------- next part --------------<br>
> An HTML attachment was scrubbed...<br>
> URL: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e7f725e6/attachment.htm" target="_blank">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e7f725e6/attachment.htm</a>><br>
><br>
> ------------------------------<br>
><br>
> _______________________________________________<br>
> theme-reviewers mailing list<br>
> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
> <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
><br>
><br>
> End of theme-reviewers Digest, Vol 11, Issue 92<br>
> ***********************************************<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
</blockquote></div><br>
<br></div></div>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></div> </div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br></div>
<br></div></div>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a></div> </div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></blockquote></div><br>