[theme-reviewers] Theme Reviwers

Fri May 4 06:32:43 UTC 2012

Emil,
I believe you are confusing between two terms: internal stylesheets and
inline styles <http://www.w3schools.com/css/css_howto.asp>. The "problems"
with caching apply to internal stylesheets (a.k.a. the <style
type='text/css'> declaration), not to inline styles. And there is an
opposing school of thought regarding internal stylesheets, one that states
that an internal stylesheet saves a HTTP request round-trip (a saving that
significantly speeds up your page load particularly if your stylesheets are
small). In any case, the mitigation for certain uses of inline styles is
much more load-intensive than the use of the inline styles themselves.

Sayontan.

On Thu, May 3, 2012 at 11:02 PM, Emil Uzelac <emil at themeid.com> wrote:

> *Of course not, we're here to discuss not to debate :) *
>
> Being an admin or reviewer has very little to do with what could be good
> or not so good for WP users. Inline styles was something developers liked
> to do I would say 6-8 years ago. Using inline styles for personal projects
> is one thing, passing this onto users is another story. If and when we
> decide to use inline styles, that's called "dictating" and that's not what
> we should do. Users should be able to change their CSS elements from
> stylesheet and not wondering where other styles might be.
>
> *In my understanding inline styles were not welcomed even before I've
> joined the team in December of 2010 :)*
>
> Other issues of inline styles:
>
>    - Code Bloating
>    - Potentially slowing down the page load time
>
> There's the reason why e.g. Google PageSpeed suggests that we load styles
> at the top of the page, styles should load before the page does, that will
> not be the case if inline styles are used. Inline styles will also cause
> browser interruption as well. Next would be that when using stylesheet as
> e.g style.css browser could easily cache that and reuse what you already
> loaded the first time you entered example.com and again inline will not.
> I can go on and on about this, but that would not be necessary :)
>
> As far as jQuery inline styles that too is not the best practice and what
> I call "CSS Hacking". With jQuery you can use the stylesheet too. That's
> why some jQuery plugins will affect the performance of the page load.
>
> I would highly recommend something like this:
>
>    - jquery.example.js
>    - jquery.example.css
>
> Cleaner code = less issues for users, less issues for users = less support
> for developers, simply stuff really.
>
> Yes, this specifically was not in any of the links I pasted above, but I
> think that it goes into
> http://codex.wordpress.org/Theme_Review#Code_Quality
>
> Anyone else if free to "jump in" and tell all of us why we should allow
> inline CSS. This is an open topic and there's nothing one admin can do if
> others don't agree with him, don't think of me that way please.
>
> Thanks,
> Emil
>
>
> On Thu, May 3, 2012 at 11:55 PM, Sayontan Sinha <sayontan at gmail.com>wrote:
>
>> Emil,
>> I don't want to get into a debate here - this is too small a point. Just
>> note this:
>>
>>    1. I am not saying inline styles are good. I am saying they are okay,
>>    depending on the context. They certainly aren't bad or wrong, and they have
>>    their place. Pick up any reliable resource on the web - they will recommend
>>    against inline styles with a caveat, that they shouldn't be used *if*you want to offer the ability to override them. One way to interpret this
>>    is, "They are fine if you don't care about the ability to override them".
>>    In the cases that I use them, I make sure that the user has no reason to
>>    override them. Correctness here is really a question of context.
>>    2. None of the links you have provided says inline styles must not be
>>    used - you can go through them. Thus it is not the official position of WP.
>>    Frankly I would be surprised if this is made an official position, in which
>>    case using the "css" function of JQuery should be disallowed too, in favour
>>    of "addClass".
>>
>> I have never officially reviewed a theme, however I do follow this thread
>> very closely. You are a WPTRT admin and thus your word carries weight
>> amongst other reviewers. It is just that I have coded some insanely complex
>> scenarios and I have come up against more "exception" situations than most
>> other developers. I am just trying to bring to light some such scenarios.
>>
>> Sayontan.
>>
>> PS: I take the "lazy" developers as a compliment. As per Larry Wall (the
>> creator of Perl), "The three chief virtues of a programmer are: Laziness,
>> Impatience and Hubris".
>>
>> On Thu, May 3, 2012 at 7:11 PM, Emil Uzelac <emil at themeid.com> wrote:
>>
>>> There's no shortcuts in CSS, saving time -vs doing it right the first
>>> time is completely different. It is very important <div
>>> style="margin-top:33px;"> will be hard to override in a stylesheet and yes
>>> it is wrong and bad practice, how can this be good. Tables too, I did not
>>> use tables in years! <div style="margin-top:33px;"> is for "lazy"
>>> developers I am sorry, not sure if you're using them or not.
>>>
>>> This is not something that I am pushing, it was much before me. My first
>>> message was also intended for reviewers and only as a reminder that's all.
>>>
>>> See:
>>>
>>>    - http://codex.wordpress.org/CSS_Coding_Standards
>>>    - http://codex.wordpress.org/WordPress_Coding_Standards
>>>
>>> second link will give you better idea how strict the WP Standards are
>>> and how much of slack review team is giving to all of us.
>>>
>>> Thanks,
>>> Emil
>>> _______________________________________________
>>>  theme-reviewers mailing list
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>>
>> --
>> Sayontan Sinha
>> http://mynethome.net | http://mynethome.net/blog
>> --
>> Beating Australia in Cricket is like killing a celebrity. The death gets
>> more coverage than the crime.
>>
>>
>> _______________________________________________
>> 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
>
>

-- 
Sayontan Sinha
http://mynethome.net | http://mynethome.net/blog
--
Beating Australia in Cricket is like killing a celebrity. The death gets
more coverage than the crime.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/pipermail/theme-reviewers/attachments/20120503/1f419903/attachment.htm>