<div dir="ltr"><a href="http://codex.wordpress.org/WordPress_Coding_Standards#Quotes">http://codex.wordpress.org/WordPress_Coding_Standards#Quotes</a><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Feb 17, 2013 at 4:09 PM, Alex Watson <span dir="ltr"><<a href="mailto:alex@alexwatson.co.uk" target="_blank">alex@alexwatson.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
Hi all
</div><div><br></div><div>I'm reviewing a theme where on the whole they don't use quotes for class/ID's - <div id=main> for example. This is fine, as far as I'm aware, but on some pages they have mixed this with class names that do use quotes <h3 class="green"> for example. Now I know none of this will stop their theme passing the review, but should I recommend that they keep things consistent and stick to one or the other?</div>
<div><br></div><div>Secondly, their code indentation isn't great. In fact on some pages its completely awful. Again this doesn't matter, but I'm a bit of a perfectionist when it comes to writing code myself, and seeing badly indented code really bugs me! It also makes it harder to read the code.</div>
<div><br></div><div>What do the rest of you do when you encounter either of the above? Just let it slide because it doesn't affect how the theme functions?</div><div><br></div><div>Many thanks</div><span class="HOEnZb"><font color="#888888"><div>
<br></div><div>Alex</div>
<div><div><br></div><div><br></div></div>
</font></span><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>