<font color="#333333"><font size="2"><font face="arial,helvetica,sans-serif">Sure <a href="http://themes.trac.wordpress.org/ticket/3498">http://themes.trac.wordpress.org/ticket/3498</a><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 Tue, Apr 19, 2011 at 6:11 PM, Tammie Lister <span dir="ltr"><<a href="mailto:tlister@logicalbinary.com">tlister@logicalbinary.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<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><a href="http://themes.trac.wordpress.org/ticket/3487" target="_blank">http://themes.trac.wordpress.org/ticket/3487</a> 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, <a href="mailto:theme-reviewers-request@lists.wordpress.org" target="_blank">theme-reviewers-request@lists.wordpress.org</a> 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" 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: <<a href="mailto:COL102-W23982B423D7C0CA1596795AE900@phx.gbl" target="_blank">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" 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: For Women-Female - 1.1<br><br>----Emil Uzelac | 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><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" 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><a href="http://WordPress.org" target="_blank">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" 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: <<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>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+CK-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><blockquote type="cite"><div> 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><a href="http://WordPress.org" target="_blank">WordPress.org</a> 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>
</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" target="_blank">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" 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: <<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>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+EbjW9CVjQ3BOcsmrZ4MAVXaQ@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 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" 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: <<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>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>>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" 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></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" target="_blank">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" 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>
</div></div></span>
</blockquote>
<div>
<br>
</div>
</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>