<div dir="ltr">Thanks a lot Chip and Other Reviewers. This conversation is really very helpful to me.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 6:54 PM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net" target="_blank">chip@chipbennett.net</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">Basically, anything "custom" that gets put in the document head should be added via callback to an appropriate hook, with wp_head being the fallback. For most scripts/stylesheets, the appropriate hook is going to be wp_enqueue_scripts. For IE-conditional scripts, this hook can be used, if the $GLOBALS['wp_styles']->add_data() method is used:<div>
<a href="http://kuttler.eu/post/wordpress-style-version-conditional-comments/" target="_blank">http://kuttler.eu/post/wordpress-style-version-conditional-comments/</a><br></div><div><br></div><div>Otherwise, wp_head can still be used.</div>
<div><br></div><div>Regarding $content_width: all functional code in functions.php should be wrapped in a callback and hooked into an appropriate action - in this case, after_setup_theme.</div></div><div class="HOEnZb"><div class="h5">
<div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Jan 21, 2013 at 8:41 AM, Bernard Latanowicz <span dir="ltr"><<a href="mailto:bernard.latanowicz@gmail.com" target="_blank">bernard.latanowicz@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div text="#000000" bgcolor="#FFFFFF">
Hi Qamar<br>
I definitely agree with about the 2nd point. There should be a
built-in function for it.<br>
Unfortunatelly I can't say a word about $content_width, because I
feel I don't understand it well.<br>
<br>
Regards,<br>
Bernard<br>
<br>
<div>W dniu 2013-01-21 14:34, Qamar Ashraf
pisze:<br>
</div>
<blockquote type="cite"><div><div>
<div dir="ltr">
<div>Hello Admin,</div>
<div>I need a final call on the following topics as there are
different views by different reviewers,</div>
<div><br>
</div>
<div><strong>1. About $content_width</strong></div>
<div><br>
</div>
<div>What is the best place for the $content_width</div>
<div><br>
</div>
<div>A. Wrap $content_width definition in a callback, hooked
into after_setup_theme - ( I am reviewing on this point )</div>
<div>B. Placing $content_width outside a function </div>
<div><br>
</div>
<div><strong>2. About IE Conditional Tags / Scripts like HTML 5
Shiv</strong></div>
<div><br>
</div>
<div>A. - Additional scripts should not be hard-coded in the
header template. The most appropriate way to handle them to
place it in a function that hooks into wp_head. ( I am
reviewing on this point )</div>
<div>B. Can be placed directly in header.php</div>
<div><br>
</div>
<div>Waiting for response.</div>
<br>
-- <br>
<font color="#666666">Regards</font>
<div><font color="#666666">Qamar Ashraf</font></div>
<div><font color="#666666">Project Manager</font></div>
<div><font color="#666666"><a href="http://www.tutorialchip.com" target="_blank">www.tutorialchip.com</a></font></div>
<div><font color="#666666"><a href="http://www.freestockphotosclub.com" target="_blank">www.freestockphotosclub.com</a></font></div>
<div><a href="http://www.twunfollowapp.com" target="_blank">www.twunfollowapp.com</a></div>
<div><font color="#666666">@lifeobject1</font></div>
</div>
<br>
<fieldset></fieldset>
<br>
</div></div><div><pre>_______________________________________________
theme-reviewers mailing list
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.wordpress.org</a>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</div></blockquote>
<br>
</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></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>