I use 10 personally, just because it's a nice, round number. And I'm an engineer, so I naturally think in orders of magnitude. :)<div><br></div><div>At least for the time being (given the nature of the core implementation), I'm fine with allowing Themes to support the default number of columns, and if necessary to document the limitation in the Theme documentation.</div>
<div><br></div><div>Getting a fix in core is probably a minimum workaround. However, I still would much rather see us get to the point where core isn't adding in arbitrary styling for one arbitrary component (galleries), when <i>core doesn't define markup for absolutely anything else</i> on the front end.</div>
<div><br></div><div>The excuse I got in the ticket was that making the change (to remove the inline styles added for the gallery shortcode) would "break" existing Themes. I'd rather see that battle fought and won, but if not, then the next alternative would be to add in core styling for up to 9 columns.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Sat, Jul 16, 2011 at 3:59 PM, Michael Fields <span dir="ltr"><<a href="mailto:michael@mfields.org">michael@mfields.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
> I had actually forgotten about that dropdown in the Media/Gallery modal window. That is actually a rather compelling argument for Twenty Eleven, at the very least, to support up to 9 columns.<br>
<br>
</div>I kinda thought so :) I always wondered where you got the number 10 from ...<br>
<div class="im"><br>
> The problem I see with allowing Themes to specify the number of columns supported, in a way that impacts that dropdown, is that it would cause unexpected behavior for pre-existing galleries that might have specified more columns than the number for which the Theme indicates support.<br>
<br>
</div>Great point! Thing is that unexpected behavior occurs when the theme does not support the columns defined as well. Perhaps the best route would be to work out css that would be generated by core and added to the styles already printed. This could take the burden completely off of the theme author while respecting the users wishes.<br>
<div class="im"><br>
On Jul 16, 2011, at 1:41 PM, Chip Bennett wrote:<br>
<br>
> I had actually forgotten about that dropdown in the Media/Gallery modal window. That is actually a rather compelling argument for Twenty Eleven, at the very least, to support up to 9 columns.<br>
><br>
> The problem I see with allowing Themes to specify the number of columns supported, in a way that impacts that dropdown, is that it would cause unexpected behavior for pre-existing galleries that might have specified more columns than the number for which the Theme indicates support.<br>
<br>
</div><div><div></div><div class="h5">_______________________________________________<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>
</div></div></blockquote></div><br></div>