Where my thinking is on this one:<div><br></div><div>The only really *critical* criterion is that Pages don&#39;t display the message, because of the (unfortunately coined) &quot;WordPress as CMS&quot; use case. Everything else, I *think* can perhaps be considered as design intent - unless there is a compelling UX consideration?</div>
<div><br></div><div>What I would like to *avoid* is frequently changing guidelines in this area - so if we want to change the guidelines now, let&#39;s nail down rock-solid reasoning for whatever we come up with, and get it right the first time.</div>
<div><br></div><div>Personally, I&#39;m hesitant to make anything RECOMMENDED, without a clear consensus. I&#39;m in favor of recommending to display &quot;comments are closed&quot; on Posts, with or without existing comments. People are acclimated to blog posts having a comment form, so such a message would basically always make sense.</div>
<div><br></div><div>So, let&#39;s start here, and build from it:</div><div><br></div><blockquote style="margin: 0 0 0 40px; border: none; padding: 0px;"><div>REQUIRED</div><div>* Do not display &quot;comments are closed&quot; on Pages (with no comments?)</div>
<div><br></div><div>OPTIONALLY</div><div>* Display &quot;comments are closed&quot; on Posts (with or without comments?)</div><div>* Display &quot;comments are closed&quot; on Posts and Pages with existing comments</div></blockquote>
<div><br></div><div>How would you (all) write them differently, and why?</div><div><br></div><div>(On a related note: this discussion will likely lead to some changes to the Theme Unit Tests, to account for all use-case permutations: Posts and Pages with and without comments, and with comments enabled/disabled.)</div>
<div><br></div><div>(On an unrelated note: I would MUCH rather see these use cases rolled into filters for core functions, i.e. comment_form(), rather than cobbled together like this.)</div><div><br></div><div>Chip<br><br>
<div class="gmail_quote">On Thu, Oct 6, 2011 at 3:04 PM, Justin Tadlock <span dir="ltr">&lt;<a href="mailto:justin@justintadlock.com">justin@justintadlock.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
We&#39;ve had this discussion before.  Might need to drag up that old thread.  Here&#39;s basically how I do it:<br>
<br>
* If the post has comments but comments are disabled, display a message.<br>
* If the post does not have comments and comments are disabled, display nothing.<br>
<br>
&quot;Post&quot; meaning posts of any post type, including pages.<div><div></div><div class="h5"><br>
<br>
On 10/5/2011 3:02 PM, Ian Stewart wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2011-10-05, at 2:59 PM, Chip Bennett wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I would even throw out for consideration the idea that displaying such a notice at all might be best left as a design decision - at least, on Blog Posts<br>
</blockquote>
Well, I do think it&#39;s helpful on posts that have comments and then — What the hey now! Where&#39;s that comment form so I can add a comment? Oh, &quot;Comments are disabled.&quot; Very well then.<br>
<br>
Ian<br>
<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
<br>
<br>
</blockquote>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</div></div></blockquote></div><br></div>