<div dir="ltr">Great. Will check in. Thanks<br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 12, 2014 at 7:23 AM, Justin Tadlock <span dir="ltr"><<a href="mailto:justin@justintadlock.com" target="_blank">justin@justintadlock.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span class="">On Thu, Sep 11, 2014 at 6:22 PM, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>></span> wrote:</span><span class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div>IMO, the title should look like this, in the header.php, in all themes:</div><div><br></div><div><title><?php wp_title(whatever); ?></title></div><div><br></div><div>I honestly can't think of any good reason to do it any other way. Certainly other ways exist, but they're all slower, make things more difficult, and harder for people to understand. </div><div><br></div><div>Why reinvent the wheel and make it octagonal?</div></div></div></div></div></blockquote><div><br></div></span><div>There are certainly different schools of thought on how to develop themes.  Some folks even like octagonal wheels. This is something I can work around and am not going to worry too much about.</div><span class=""><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span><div>Actually, that check just sees if register_sidebar() is in the theme and if widgets_init is not in the theme. If you are registering a sidebar, then it should be happening in a function hooked to widgets_init, and not anywhere else. Doesn't matter how it's handling HTML5 or what have you. <br></div></span></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">So if the theme doesn't contain "widgets_init" anywhere but still registers sidebars, that's a legitimate issue.</div></div></blockquote><div><br></div></span><div>The issue is that the theme doesn't register widgets but the framework for the theme has a wrapper function for register_sidebar().  The call to register_sidebar() exists within the framework.  Here's a simplified version of the code:<br></div><div><br></div><div><span style="white-space:pre-wrap"> </span>function prefix_register_sidebar( $args ) {</div><div><span style="white-space:pre-wrap">              </span>/* set some defaults up. */</div><div><span style="white-space:pre-wrap">              </span>register_sidebar( $args );</div><div><span style="white-space:pre-wrap">       </span>}</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></div>