[theme-reviewers] Need A Final Call

[Chip Bennett]

Yep, been watching that ticket for almost a year and a half. :)


On Mon, Jan 21, 2013 at 9:21 AM, Sami Keijonen <sami.keijonen at foxnet.fi>wrote:

> And it seems like IE conditional loading for js is finally gonna happen.
> This is useful in HTML 5 Shiv.
>
> http://core.trac.wordpress.org/ticket/16024
>
> 2013/1/21 Qamar Ashraf <life.object at gmail.com>
>
>> Hello Admin,
>> I need a final call on the following topics as there are different views
>> by different reviewers,
>>
>> *1. About $content_width*
>>
>> What is the best place for the $content_width
>>
>> A. Wrap $content_width definition in a callback, hooked into
>> after_setup_theme - ( I am reviewing on this point )
>> B. Placing $content_width outside a function
>>
>> *2. About IE Conditional Tags / Scripts like HTML 5 Shiv*
>>
>> 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 )
>> B. Can be placed directly in header.php
>>
>> Waiting for response.
>>
>> --
>> Regards
>> Qamar Ashraf
>> Project Manager
>> www.tutorialchip.com
>> www.freestockphotosclub.com
>> www.twunfollowapp.com
>> @lifeobject1
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
>
> --
> Sami Keijonen
> Fox Holding Oy
> sami.keijonen at foxnet.fi
> www.foxnet.fi
>
> _______________________________________________
> theme-reviewers mailing list
> theme-reviewers at lists.wordpress.org
> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20130121/d77d3e9f/attachment.htm>