<div dir="ltr">No, this is not correct per the Guidelines, which require $content_width, along with all other functional output, to be called within a callback hooked into an appropriate hook.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Wed, Jun 18, 2014 at 7:05 AM, Konstantin Kovshenin <span dir="ltr"><<a href="mailto:kovshenin@gmail.com" target="_blank">kovshenin@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Technically it doesn't really matter. after_setup_theme runs right<br>
after the functions.php file has been loaded, so declaring it in the<br>
global scope is just as good. Make sure it's wrapped into an isset()<br>
call though, so child themes can override the setting.<br>
<div class="HOEnZb"><div class="h5"><br>
On Wed, Jun 18, 2014 at 3:01 PM, Ola Łączek <<a href="mailto:ola@bodera.com">ola@bodera.com</a>> wrote:<br>
> Hello!<br>
><br>
> I've got a question about $content_width. I remember reading somewhere that<br>
> theme authors should include it in main setup function. Is that right? When<br>
> I checked Twenty Fourteen it has $content_width declared outside setup<br>
> function. I can't seem to find it the answer in the guidelines, so I would<br>
> appreciate your help.<br>
><br>
> Best regards,<br>
> Ola<br>
><br>
</div></div><div class="im HOEnZb">> _______________________________________________<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>
<br>
<br>
<br>
</div><span class="HOEnZb"><font color="#888888">--<br>
Konstantin<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div>