Ignore this part:<div><br></div><blockquote class="webkit-indent-blockquote" style="margin: 0 0 0 40px; border: none; padding: 0px;"><div><div>Note: the Theme also fails the license requirement, due to this line in style.css:</div>
<div><br></div><blockquote style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 40px; border-top-style: none; border-right-style: none; border-bottom-style: none; border-left-style: none; border-width: initial; border-color: initial; padding-top: 0px; padding-right: 0px; padding-bottom: 0px; padding-left: 0px; ">
<div><div><i>If you remix, convert, translate, modify, or distribute a copy of this theme, please keep the links/credit in the footer.</i></div></div></blockquote></div></blockquote><div><br></div><div>I had two style.css documents open. That line comes from the one I'm currently reviewing.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Tue, Sep 7, 2010 at 8:52 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
More specifically:<div> 1) Themes may incorporate ONE public-facing credit link</div><div> 2) Credit link must be either Theme URI or Author URI (as defined in style.css)</div><div> 3) Both Theme URI and Author URI must be accurate and relevant.</div>
<div><br></div><div>Credit Links are a *critical* requirement, IMHO. I would require an appropriate URI be provided in all of the following locations:</div><div> * style.css Theme URI</div><div> * style.css Description</div>
<div> * footer.php credit link (and only one such link is permitted. Theme has 2)</div><div><br></div><div>Note: the Theme also fails the license requirement, due to this line in style.css:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px">
<div><div><i>If you remix, convert, translate, modify, or distribute a copy of this theme, please keep the links/credit in the footer.</i></div></div></blockquote><div><br></div><div>FYI, License and Credit Links are the first two things I check for any Theme. This is why, because it's (unfortunately) all too common.</div>
<div><br></div><div>Further, use of text string "twentyten" as the Theme's textdomain is an automatic fail. Text domains must be globally unique. I would also include as a *strongly recommended, not required* for the Theme to replace "twentyten_" function prefixes with a unique prefix (preferably, the Theme slug), and to replace all inline-documentation references to TwentyTen (except as applicable).</div>
<div><br></div><div>Chip</div><div><br><div class="gmail_quote"><div><div></div><div class="h5">On Tue, Sep 7, 2010 at 8:07 AM, Chris <span dir="ltr"><<a href="mailto:chris@thematic4you.com" target="_blank">chris@thematic4you.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5">
<div bgcolor="white" lang="DE" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Hi Devin,</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">I would reject that theme for the <a href="http://shopedia.se" target="_blank">shopedia.se</a> link. The links
need to be author site / theme page. </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Chris</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;color:windowtext">Von:</span></b><span lang="EN-US" style="font-size:10.0pt;color:windowtext">
<a href="mailto:theme-reviewers-bounces@lists.wordpress.org" target="_blank">theme-reviewers-bounces@lists.wordpress.org</a>
[mailto:<a href="mailto:theme-reviewers-bounces@lists.wordpress.or" target="_blank">theme-reviewers-bounces@lists.wordpress.or</a></span><span style="font-size:10.0pt;color:windowtext">g] <b>Im
Auftrag von </b>Devin Price<br>
<b>Gesendet:</b> Dienstag, 7. September 2010 05:41<br>
<b>An:</b> <a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
<b>Betreff:</b> [theme-reviewers] Modern Green</span></p>
</div>
</div><div><div></div><div>
<p class="MsoNormal"> </p>
<p class="MsoNormal">Every theme I've reviewed so far has been an easy fail, but
this one looks good to go. Can someone check my work and make sure it
actually does pass review: <a href="http://themes.trac.wordpress.org/ticket/943" target="_blank">http://themes.trac.wordpress.org/ticket/943</a>
? I assigned it as "needs-additional-review", but it
disappeared from the queue. Also, I wasn't sure about the link (<a href="http://shopedia.se/" target="_blank">http://shopedia.se/</a>) if that actually needs to
go to a theme page or actual author site. Thanks.</p>
<div>
<p>Devin Price<br>
<a href="http://www.wptheming.com" target="_blank">WP Theming Inc.</a><br>
Twitter: <a href="http://twitter.com/devinsays" target="_blank">@devinsays</a></p>
</div>
</div></div></div>
</div>
</div>
<br></div></div>_______________________________________________<br>
theme-reviewers mailing list<div class="im"><br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a><br>
</div><div class="im"><a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
<br></div></blockquote></div><br></div>
</blockquote></div><br></div>