<div>
<div>
<span>
<div><span>I've completed another theme review tonight and would like to review another please tomorrow taking the feedback on board of extra things I should be looking for. Again happy either BuddyPress or WordPress.</span></div><div><span><br></span></div><div><span>http://themes.trac.wordpress.org/ticket/3487 was one did tonight.</span></div><div><span><br>Regards,<div>Tammie Lister</div></span></div></span><span><div><br></div></span>
<!-- <p style="color: #a0a0a0;">On Tuesday, 19 April 2011 at 23:58, theme-reviewers-request@lists.wordpress.org wrote:</p> -->
<p style="color: #a0a0a0;">On Tuesday, 19 April 2011 at 23:58, theme-reviewers-request@lists.wordpress.org wrote:</p>
<blockquote type="cite" style="border-left-style:solid;border-width:1px;margin-left:0px;padding-left:10px;">
<span><div><div>Send theme-reviewers mailing list submissions to<br> <a href="mailto:theme-reviewers@lists.wordpress.org">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">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">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">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">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">theme-reviewers@lists.wordpress.org</a>><br>Message-ID: <<a href="mailto:COL102-W23982B423D7C0CA1596795AE900@phx.gbl">COL102-W23982B423D7C0CA1596795AE900@phx.gbl</a>><br>Content-Type: text/plain; charset="iso-8859-1"<br><br><br>done<br><br>From: <a href="mailto:emil@themeid.com">emil@themeid.com</a><br>Date: Tue, 19 Apr 2011 15:50:22 -0500<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>Subject: Re: [theme-reviewers] @smgrundy [WordPress Themes] #3563: THEME: For Women-Female - 1.1<br><br>----Emil Uzelac | ThemeID | T: 224-444-0006 | Twitter: @EmilUzelac | E: emil@themeid.com | <a href="http://themeid.com">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 <<a href="mailto:theme-reviewers@lists.wordpress.org">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">http://themes.trac.wordpress.org/ticket/3563#comment:1</a>><br><br>WordPress Themes <<a href="http://themes.trac.wordpress.org/">http://themes.trac.wordpress.org/</a>><br><br><a href="http://WordPress.org">WordPress.org</a> Theme Directory Reviews<br><br><br><br><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">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a> <br>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>URL: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e2e4a631/attachment-0001.htm">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/e2e4a631/attachment-0001.htm</a>><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">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">theme-reviewers@lists.wordpress.org</a><br>Message-ID: <<a href="mailto:BANLkTikKrPxMtJrueT+CK-pt4BfWN1nJpA@mail.gmail.com">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>emil@themeid.com | <a href="http://themeid.com">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">smgrundy@live.com</a>> wrote:<br><br><blockquote type="cite"><div> done<br><br>------------------------------<br>From: <a href="mailto:emil@themeid.com">emil@themeid.com</a><br>Date: Tue, 19 Apr 2011 15:50:22 -0500<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org">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>emil@themeid.com | <a href="http://themeid.com">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">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">http://themes.trac.wordpress.org/ticket/3563#comment:1</a>><br>WordPress Themes <<a href="http://themes.trac.wordpress.org/">http://themes.trac.wordpress.org/</a>><br><a href="http://WordPress.org">WordPress.org</a> Theme Directory Reviews<br><br><br><br>_______________________________________________ theme-reviewers mailing<br>list <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">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">theme-reviewers@lists.wordpress.org</a><br><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br></div></blockquote>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>URL: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/5999840f/attachment-0001.htm">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/5999840f/attachment-0001.htm</a>><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">rexposadas@yahoo.com</a>><br>Subject: [theme-reviewers] After 3367 - request<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>Message-ID: <<a href="mailto:969894.75830.qm@web36504.mail.mud.yahoo.com">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: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/96396658/attachment-0001.htm">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/96396658/attachment-0001.htm</a>><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">curtis@curtismchale.ca</a>><br>Subject: Re: [theme-reviewers] Another ticket<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>Message-ID: <BANLkTi==<a href="mailto:o+EbjW9CVjQ3BOcsmrZ4MAVXaQ@mail.gmail.com">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">http://browser-update.org</a>/). There is not a theme option 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 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 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">http://www.curtismchale.ca</a><br>Linkedin: <a href="http://www.linkedin.com/in/curtismchale">http://www.linkedin.com/in/curtismchale</a><br>Twitter: <a href="http://twitter.com/curtismchale">twitter.com/curtismchale</a><br>-------------- next part --------------<br>An HTML attachment was scrubbed...<br>URL: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/9b218d3d/attachment-0001.htm">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/9b218d3d/attachment-0001.htm</a>><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">chip@chipbennett.net</a>><br>Subject: Re: [theme-reviewers] Another ticket<br>To: <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>Message-ID: <BANLkTi=<a href="mailto:Z_Qb3E9msQQNLhMAr70_Hrk5CmA@mail.gmail.com">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">curtis@curtismchale.ca</a>>wrote:<br><br><blockquote type="cite"><div>The theme I'm reviewing has JS that detects the browser version and tells<br>them to update (<a href="http://browser-update.org">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></div></blockquote><br>Incredibly annoying, but nothing prohibits it. I would *require* a mention<br>of the "feature" in Theme documentation (readme.txt).<br><br><blockquote type="cite"><div><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></div></blockquote><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><blockquote type="cite"><div><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></div></blockquote><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><blockquote type="cite"><div><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></div></blockquote><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><blockquote type="cite"><div><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></div></blockquote><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: <<a href="http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/01dbdb66/attachment.htm">http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20110419/01dbdb66/attachment.htm</a>><br><br>------------------------------<br><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">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br><br><br>End of theme-reviewers Digest, Vol 11, Issue 90<br>***********************************************<br></div></div></span>
</blockquote>
<div>
<br>
</div>
</div>
</div>