<font color="#333333"><font><font face="arial,helvetica,sans-serif">Hi Bill,</font></font></font><div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">You're not out of the line, this code is available to the public and if you can come up with the better solution please go right ahead. Codes were not written by me, all I did was to make them <a href="http://codex.wordpress.org/Theme_Review">Theme Review</a> compatible and to help authors getting their images properly aligned, that's pretty much it. They are not there just as an example, that's a working CSS copy used on who knows how many Themes.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">On other hand I personally did not find this to be unnecessary because not every developer will have clean codes and Theme styles would override generated classes easily, that would be one of the reasons for having DIV element in front of the property and if they don't do anything, but it can help under special circumstances, why remove them? All declarations are there for a reason :)</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">As far as margins for alignnone, why not? it looks much nicer and cleaner too, but we're also jumping from what's really needed to a personal preferences.</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Thanks,</font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif">Emil</font></font></font></div>
<div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font></div><div><font color="#333333"><font><font face="arial,helvetica,sans-serif"><br></font></font></font><br><div class="gmail_quote">
On Wed, Jul 18, 2012 at 4:19 PM, William Ludwig <span dir="ltr"><<a href="mailto:weludwig@gmail.com" target="_blank">weludwig@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>Okay, I'm normally quiet and tend to defer to those who have more experience but I have a slight concern about pointing people to the sample CSS provided for alignments at <a href="http://codex.wordpress.org/CSS#WordPress_Generated_Classes" target="_blank">http://codex.wordpress.org/CSS#WordPress_Generated_Classes</a></div>
<div><br></div><div>The CSS provided, while functional, is far from effecient and I would worry about developers copy/pasting that wholesale into their theme without editing. For example the second rule has both .aligncenter and div.aligncenter as selectors. The div.aligncenter is unnecessary and adds nothing. There are several other rules below that which are just slower selectors with the same rules as above.</div>
<div><br></div><div>I know that this is just meant as examples to get someone started but if I saw this in a theme I would really question the person who wrote it.</div><div><br></div><div>To the original question I would suggest not adding any alignment/margin/etc to images with alignnone. The way I have always understood it was that alignnone images should display inline with the surrounding content naturally.</div>
<div><br></div><div>I hope I'm not out of line and I reserve the right to be completely wrong. Thanks.</div><div>Bill Ludwig</div><div class="HOEnZb"><div class="h5"><div><br></div><br><div class="gmail_quote">On Wed, Jul 18, 2012 at 4:17 PM, esmi at quirm dot net <span dir="ltr"><<a href="mailto:esmi@quirm.net" target="_blank">esmi@quirm.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">on 18/07/2012 21:46 Emil Uzelac said the following:<div><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
While ago I rewrote<br>
<a href="http://codex.wordpress.org/CSS#WordPress_Generated_Classes" target="_blank">http://codex.wordpress.org/<u></u>CSS#WordPress_Generated_<u></u>Classes</a> at the moment<br>
that was (IMO) the best align* practices. Clear isn't the best thing to do,<br>
because that will create an issue down the road.<br>
</blockquote>
<br></div>
I was unsure about the clearing too but some default margins would be a good idea. I'll start adding this into my themes.<br>
<br>
Perhaps that Codex sub-section could be added to the Resources on <<a href="http://make.wordpress.org/themes" target="_blank">http://make.wordpress.org/<u></u>themes</a>>?<div><div><br>
<br>
Mel<br>
-- <br>
<a href="http://quirm.net" target="_blank">http://quirm.net</a><br>
<a href="http://blackwidows.co.uk" target="_blank">http://blackwidows.co.uk</a><br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</div></div></blockquote></div><br>
</div></div><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>