[theme-reviewers] Need clarification on Guideline.

[Emil Uzelac]

Very true @import is not the greatest performance wise. However
loading more than one CSS is too. That's why my practice is to
load as much as I can into a single style.css

Remember, first @import will start with a Child Theme.


On Tue, Jul 16, 2013 at 5:52 PM, Stephen Cui <scui2005 at gmail.com> wrote:

> Thanks for the clarification. My question is pure Guideline clarification.
>
> I understand what is best practice. For example. each @import is a HTTP
> request. Therefore, each import of internal stylesheet will have
> performance consequence. Am I right?
>
>
> On Tue, Jul 16, 2013 at 5:33 PM, Chip Bennett <chip at chipbennett.net>wrote:
>
>> And also, just to note: @import or enqueueing an external stylesheet is
>> extremely limited. It should only be used for a trusted, third-party API
>> like Google Fonts.
>>
>> Any stylesheets that *can* be bundled with the Theme *should* be, rather
>> than enqueued externally.
>>
>> But, as Emil said: in this case, whether you @import or enqueue, either
>> should be just fine.
>>
>>
>> On Tue, Jul 16, 2013 at 6:25 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>
>>> Maybe this will explain better.
>>>
>>> Needs to be enqueued:
>>>
>>> <link href='http://fonts.googleapis.com/css?family=Fauna+One'
>>> rel='stylesheet' type='text/css'>
>>>
>>> Does not need to be:
>>>
>>> @import url(http://fonts.googleapis.com/css?family=Fauna+One);
>>>
>>> If handled from style.css
>>>
>>> For the references how to include Google Fonts in i.e. functions.php
>>> please see
>>> http://themes.svn.wordpress.org/babylog/2.1/functions.php
>>>
>>>
>>> On Tue, Jul 16, 2013 at 5:16 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>>
>>>> No sir, it does not need to be.
>>>>
>>>>
>>>> On Tue, Jul 16, 2013 at 5:15 PM, Stephen Cui <scui2005 at gmail.com>wrote:
>>>>
>>>>> Ok. But google font need to be enqueued?
>>>>>
>>>>> Stephen
>>>>>
>>>>>
>>>>> On Jul 16, 2013, at 5:07 PM, Emil Uzelac <emil at uzelac.me> wrote:
>>>>>
>>>>> @import's are actually fine, if they're in the style.css no need to
>>>>> enqueue.
>>>>>
>>>>>
>>>>> On Tue, Jul 16, 2013 at 5:05 PM, Stephen Cui <scui2005 at gmail.com>wrote:
>>>>>
>>>>>> Based on the guidelines:
>>>>>>
>>>>>> Themes are *required* to enqueue all stylesheets and scripts.
>>>>>>
>>>>>> Example:
>>>>>>
>>>>>> @import url(http://fonts.googleapis.com/css?family=Cuprum&subset=latin);
>>>>>> @import url(media/css/my/reset.css);
>>>>>> @import url(media/css/settings.css);
>>>>>> @import url(media/css/widget.css);
>>>>>> @import url(media/css/comment.css);
>>>>>>
>>>>>> I believe the example here is against the guideline. Correct me if I am wrong.
>>>>>>
>>>>>> Regards
>>>>>>
>>>>>> Stephen
>>>>>>
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> theme-reviewers mailing list
>>>>>> theme-reviewers at lists.wordpress.org
>>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>>
>>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing list
>>>>> theme-reviewers at lists.wordpress.org
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> theme-reviewers mailing list
>>>>> theme-reviewers at lists.wordpress.org
>>>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>>>
>>>>>
>>>>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
> _______________________________________________
> 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/20130716/87c2c840/attachment.html>