Quite possibly. But it is not the responsibility of Themes to account for content added by Plugins.<div><br></div><div>I see no reason to require Themes to support a non-core &#39;tweetback&#39; comment-type. If a Plugin adds this comment-type, then the Plugin should be responsible for incorporating it.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Tue, Apr 26, 2011 at 3:14 PM, Sayontan Sinha <span dir="ltr">&lt;<a href="mailto:sayontan@gmail.com">sayontan@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Chip,<br>I believe Justin is referring to the fact that plugins can add the type &quot;tweetback&quot;. If that is the case, then a theme that is explicitly checking only for &quot;comment&quot;, &quot;pingback&quot; and &quot;trackback&quot; is missing out on the ones that don&#39;t fall into these buckets, i.e. it is missing a catch-all for types introduced by plugins.<br>

<br>Sayontan.<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Tue, Apr 26, 2011 at 12:49 PM, Chip Bennett <span dir="ltr">&lt;<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I can&#39;t find that &#39;tweetback&#39; is a core comment type.<div><br></div><div><a href="http://codex.wordpress.org/Function_Reference/wp_list_comments" target="_blank">According to the Codex</a>, the valid types are: <span style="font-family:&#39;Lucida Grande&#39;, Verdana, &#39;Bitstream Vera Sans&#39;, Arial, sans-serif;font-size:12px;line-height:22px">&#39;all&#39;, &#39;comment&#39;, &#39;trackback&#39;, &#39;pingback&#39;, or &#39;pings&#39;</span></div>


<div><br></div><div>So, if a Theme accounts for these types, that should be sufficient. For instance, if a Theme accounts for &#39;comments&#39; and &#39;pings&#39;, all bases are covered.</div><div><br></div><div><font color="#888888">Chip</font><div>

<div></div><div><br>
<br><div class="gmail_quote">On Tue, Apr 26, 2011 at 2:39 PM, Chip Bennett <span dir="ltr">&lt;<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Hmm... I don&#39;t think I&#39;ve yet seen a Theme that explicitly handles tweetbacks. (Honestly, I didn&#39;t even realize such a comment type existed.)<div><br></div><div><font color="#888888">Chip</font><div><div></div>


<div><br><br><div class="gmail_quote">On Wed, Apr 27, 2011 at 2:16 PM, Justin Tadlock <span dir="ltr">&lt;<a href="mailto:justin@justintadlock.com" target="_blank">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">Here&#39;s a few things we should be on the lookout for when reviewing themes that I thought I&#39;d bring up.<br>
<br>
The use of the_post_thumbnail() with the_content() can sometimes be a problem.  If a user places the image within the post content (at the beginning of the post) and sets the same image as the &quot;feature image,&quot; it creates a duplicate image issue.  Some themes&#39; designs are meant to handle this while others aren&#39;t.<br>




<br>
Some themes have a comments callback function where they don&#39;t recognize comment types other than &#39;comment&#39;, &#39;pingback&#39;, and &#39;trackback&#39;.   This is also the case in the TwentyTen theme.  If you look at its switch statement, you&#39;ll notice it doesn&#39;t give a &#39;default&#39; case.  It should be corrected to handle all comment types (e.g., tweetbacks).<br>




<br>
Loading JS and CSS on all pages of the admin.  Sometimes, themes hook their theme settings page JavaScript and Stylesheet to the &#39;admin_init&#39; hook or something similar.  This should only be loaded on the the theme settings page.  If using the add_theme_page() function, a hook is created just for that page.  A better hook would probably be &#39;load-appearance_page_$pagename&#39;.<br>




_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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>
</blockquote></div><br></div></div></div>
</blockquote></div><br></div></div></div>
<br>_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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><br clear="all"><br></div></div>-- <br>Sayontan Sinha<br><a href="http://mynethome.net" target="_blank">http://mynethome.net</a> | <a href="http://mynethome.net/blog" target="_blank">http://mynethome.net/blog</a><br>
<font color="#888888">
--<br>Beating Australia in Cricket is like killing a celebrity. The death gets more coverage than the crime.<br><br>
</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></div>