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 class="webkit-indent-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&#39;s (unfortunately) all too common.</div>
<div><br></div><div>Further, use of text string &quot;twentyten&quot; as the Theme&#39;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 &quot;twentyten_&quot; 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">On Tue, Sep 7, 2010 at 8:07 AM, Chris <span dir="ltr">&lt;<a href="mailto:chris@thematic4you.com">chris@thematic4you.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">









<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 class="h5">

<p class="MsoNormal"> </p>

<p class="MsoNormal">Every theme I&#39;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 &quot;needs-additional-review&quot;, but it
disappeared from the queue.  Also, I wasn&#39;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>_______________________________________________<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></div>