Items 1 through 3 are fine with me ... I've never been too much of a fan of item 3, but it's been long standing as a required item.<br><br>Item 4 I am undecided on, recommended woould definitely be the "strongest" wording I would accept at this time; and, if it is to be recommended (and perhaps this should apply to all recommended items) sample code (as a suggested best-practice method?) to accomplish the recommendation should be available directly from the Theme Review page if not on the page itself.<br>
<br><br>Cais.<br><br>PS: I may have more to add to thread later ... still catching up on a lot of items after a HD (pseudo-)failure. EAC<br><br><div class="gmail_quote">On Mon, Sep 13, 2010 at 12:08 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Mostly relating to the Theme Unit Tests, I would propose the following:<div><br></div><div>
1) Ignore the intent of the caption text on the centered image with caption (e.g. "must be centered, with box"). All that matters is that image and caption are centered.</div>
<div>2) Author Comment styled (e.g. .bypostauthor CSS class defined) should be *recommended*, not *required*</div><div>3) Not displaying "comments are closed" text on Pages should be *recommended*, not *required*.</div>
<div>4) Separation of track/pingbacks from comments should be *recommended*</div><div><br></div><div>Thoughts?</div><div><br></div><div>Also, anything that anyone else has come across? Anything else that you think should be downgraded from *required* to *recommended* (or vice versa, I suppose)?</div>
<div><br></div><font color="#888888"><div>Chip</div>
</font><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>