<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font size="-1"><font face="Arial">I do agree that inline styles are
definitely not best practice, but as Sayontan has pointed out
there are certain valid use cases with them. Of course, we don't
want theme developers to litter the bulk of their codes with
inline styles, but enforcing them as required would be
nitpicking on little things that would not bring significant
benefits. It will only add another barrier of entry to new theme
developers looking to contribute to the repository, and dump
additional work on existing developers who are already
stretching their time contributing to the repository. This is
especially when the majority of theme users will not even notice
a difference to their user experience as a result of this
change. Inline styles are easily overridden by using </font></font><font
size="-1"><tt>!important</tt></font><font size="-1"><font
face="Arial">, anyways.<br>
<br>
I would agree with it being recommended rather than required.
That way theme developers who are using inline styles can
gradually update their codes to get rid of most of the inline
styles. Guidelines should be that - guidelines. Every new item
being enforced as required will add additional work to existing
developers, and additional barrier of entry to new developers.
Everything comes at a cost, and for this case I do not think the
benefits outweigh the costs.<br>
</font></font>
<pre class="moz-signature" cols="72">--
Regards,
Syahir Hakim
</pre>
<br>
On 4/05/2012 8:15 p.m., Greg Priday wrote:
<blockquote
cite="mid:CALvx2c+0PRZ1_NhowES-JMubwq0XAGz+Ct2xwMKwTdzMOz5eEg@mail.gmail.com"
type="cite">
<pre wrap="">I feel there's a certain spirit of WordPress that makes the community
great and has helped it thrive.
There's nothing inherently wrong with using the style attribute here
and there, but I do feel it goes against this spirit. I'm sure most
WordPress developers will agree that using the style attribute just
doesn't feel right. To me, it just looks plain ugly - I dont even like
using them during development.
On the other hand, as a once-OO developer, I used to hate WordPress'
use of global variables all willy nilly. It's part of the spirit of
WordPress though, so I've learned to embrace, and even appreciate,
globals. So I definitely understand your views Sayontan. The spirit of
WordPress can sometimes be confusing, dogmatic, etc - but it's usually
for the best.
Time to get back to work. I'm sure we all have awesome stuff to build.
On Fri, May 4, 2012 at 10:01 AM, Sayontan Sinha <a class="moz-txt-link-rfc2396E" href="mailto:sayontan@gmail.com"><sayontan@gmail.com></a> wrote:
</pre>
<blockquote type="cite">
<blockquote type="cite">
<pre wrap="">Do we need to continue our "discussion" ? ;)
</pre>
</blockquote>
<pre wrap="">
By all means go ahead. As I said, your links give examples of repeated
styles, which I am not disputing. As I pointed out in my previous post, for
a style used only once, an inline style is less bloat, akin to the Hello
World patterns example. (I had a much longer text written about repeated vs.
non-repeated styles in one of my prior posts, but I deleted it because I
wanted to shorten the mail)
Anyway, I don't believe I am going to convince or be convinced, so this is
the end of my contribution to this thread. Feel free to thrust this as a
"REQUIRED" item for theme approval if you feel that is the best recourse. I
still believe that enforcing "No inline styles" is dogmatic and without
appropriate technical merit, but I am probably in a minority of 1.
On Fri, May 4, 2012 at 12:40 AM, Emil Uzelac <a class="moz-txt-link-rfc2396E" href="mailto:emil@themeid.com"><emil@themeid.com></a> wrote:
</pre>
<blockquote type="cite">
<pre wrap="">
And to add and possibly finish this
up: <a class="moz-txt-link-freetext" href="http://www.websiteoptimization.com/secrets/web-page/replace-inline-style.html">http://www.websiteoptimization.com/secrets/web-page/replace-inline-style.html</a>
was published in <a class="moz-txt-link-freetext" href="http://www.amazon.com/dp/0596515081/?tag=websiteoptimi-20">http://www.amazon.com/dp/0596515081/?tag=websiteoptimi-20</a>
as well ;) Do we need to continue our "discussion" ? ;)
Thanks,
Emil
</pre>
</blockquote>
<pre wrap="">
_______________________________________________
theme-reviewers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>
<a class="moz-txt-link-freetext" href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
</body>
</html>