Looks great!<div><br></div><div>My only comment regards the by-file listing for functions, tags, and hooks: I had it laid out that way to facilitate stepping through the Theme files during the review: functions.php, header.php, index.php, comments.php, sidebar.php, footer.php.</div>
<div><br></div><div>If we could organize the list such that it sorts by required/recommended/optional, and still facilitates a single-pass Theme step-through, that would be awesome. I'll take a look (later).</div><div>
<br></div><div>The next thing on my to-do list was to add Codex links, so I'll try to clean that up, also, when I get the chance.</div><div><br></div><div>Chip<br><br><div class="gmail_quote">On Sun, Aug 22, 2010 at 1:28 PM, Philip M. Hofer (Frumph) <span dir="ltr"><<a href="mailto:philip@frumph.net">philip@frumph.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div bgcolor="#ffffff">
<div><font size="2" face="Arial">Cais,</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">Revamped the Theme Review codex per our discussion
we had earliar. * Need to verify that I didn't miss any
thing.</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial"><a href="http://codex.wordpress.org/Theme_Review#WordPress_Hooks_and_Template_Tags" target="_blank">http://codex.wordpress.org/Theme_Review#WordPress_Hooks_and_Template_Tags</a></font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">Changed: Version information shows what
features were released with required implementation date.</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">Changed: Hooks and Template tags to denote a
bulk of what is must, should and other optionals instead of separating them by
individual files so it would be in a checklist type appearing.</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">* Still needs to be updated to fix all the
<tt>'s adjusting them where appropriate and implement proper
codex links. I'm not the best with wikicode.</font></div>
<div><font size="2" face="Arial"></font> </div>
<div><font size="2" face="Arial">- Phil</font></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>