I definitely think that there is a place for galleries with more than 4 columns. For galleries of more than a couple dozen photos, the horizontal space required for the gallery becomes excessive, unless the number of columns is increased (even at the expense of creating smaller thumbnails).<div>
<br></div><div>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.</div><div><br>
</div><div>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.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Sat, Jul 16, 2011 at 11:56 AM, 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;">
Just a few thoughts on this topic:<br>
<br>
Themes can style all of the gallery columns without modifying the inline css printed by the gallery shortcode.<br>
It can even be done without use of !important if the right specificity is used.<br>
Please see the following patch for twentyeleven.<br>
<br>
<a href="http://core.trac.wordpress.org/ticket/17564" target="_blank">http://core.trac.wordpress.org/ticket/17564</a><br>
<br>
The related ticket is here:<br>
<br>
<a href="http://core.trac.wordpress.org/ticket/17564" target="_blank">http://core.trac.wordpress.org/ticket/17564</a><br>
<br>
If you read the ticket, you'll see that I originally supplied a patch that supported 9 columns, but only 4 columns ended up being supported by the theme. Ian Stewart commented that supporting anything over 4 columns "seems a bit much". This is one of those situations where I agree and disagree at the same time. I mean, a 9-column gallery is something that I would never use in practice. Most themes do not even have room to do it well.<br>
<br>
Here's an example from my own theme: <a href="http://wordpress.mfields.org/test/gallery-shortcode/" target="_blank">http://wordpress.mfields.org/test/gallery-shortcode/</a> the thumbnails are tiny compared to the actual size of the image. The only galleries that display the correct image size are the ones having 1 - 4 columns. In galleries showing 5 - 9 columns, the images are resized in the browser to enable then to fit inside the theme.<br>
<br>
One possible solution that I see here is to all theme author the ability to define theme support for "gallery_columns". WordPress can then use this value to modify the number of options displayed in the "Gallery Settings" section of the modal media window.<br>
<br>
Best,<br>
-Mike<br>
<div><div></div><div class="h5"><br>
<br>
<br>
On Jul 16, 2011, at 9:30 AM, Merci Javier wrote:<br>
<br>
><br>
> At this point, what's the bottom line for required minimum number of columns supported by a theme if they override the WP gallery styles as opposed to recommended (10 columns) when reviewing a theme? 3, 4, or 5 columns?<br>
><br>
><br>
><br>
> On Sat, Jul 16, 2011 at 8:43 AM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
> I'll link the existing ticket when I'm back at my laptop; it's been wontfixed.<br>
><br>
> Chip<br>
><br>
> On 7/16/11, Edward Caissie <<a href="mailto:edward.caissie@gmail.com">edward.caissie@gmail.com</a>> wrote:<br>
> > I'm reading "needs patch" from this discussion to "correct" the method core<br>
> > is currently using.<br>
> ><br>
> > As a recommendation, I'm all for it ... if we can have core "fixed" to a<br>
> > more "correct" method or default style then I would entertain the<br>
> > possibility of pushing the recommendation to a requirement as we have done<br>
> > with other guidelines.<br>
> ><br>
> ><br>
> > Cais.<br>
> ><br>
> ><br>
> > On Fri, Jul 15, 2011 at 7:56 PM, Chip Bennett <<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>> wrote:<br>
> ><br>
> >> +1 to this:<br>
> >><br>
> >> *With that said, WP's handling of this is crap.*<br>
> >><br>
> >><br>
> >> And therein lies the problem with even considering making it a<br>
> >> requirement.<br>
> >> The underlying implementation (inline styles by default, and force Theme<br>
> >> developers to remove them via filter) impedes Theme developers - but I've<br>
> >> tried to fight that fight, and lost. So, as long as WordPress core<br>
> >> continues<br>
> >> NOT to do it right, I think it would be an unreasonable burden on Theme<br>
> >> developers to *require* them to support more than the default number of<br>
> >> columns.<br>
> >><br>
> >> Chip<br>
> >><br>
> >><br>
> >> On Sat, Jul 16, 2011 at 6:33 PM, Justin Tadlock<br>
> >> <<a href="mailto:justin@justintadlock.com">justin@justintadlock.com</a>>wrote:<br>
> >><br>
> >>> Part of me thinks it should be a requirement. WordPress already handles<br>
> >>> styles for 1 - 10 columns (and more) by default. If a theme is<br>
> >>> overwriting<br>
> >>> this with custom styles or removing the inline <style> WP adds, they need<br>
> >>> to<br>
> >>> handle this on their own. That's where the biggest problem is -- themes<br>
> >>> overwriting this but not handling anything other than 3 columns.<br>
> >>><br>
> >>> With that said, WP's handling of this is crap. I've never designed a<br>
> >>> theme where it doesn't break with the default WP styles. Of course, I<br>
> >>> always roll my own handling of this anyway.<br>
> >>><br>
> >>><br>
> >>> On 7/15/2011 5:39 PM, Ian Stewart wrote:<br>
> >>><br>
> >>>> On 2011-07-15, at 5:36 PM, Chip Bennett wrote:<br>
> >>>><br>
> >>>> Personally, I wasn't even thinking along those lines. Making something<br>
> >>>>> like that a *requirement* seems, IMHO, to cross the line into dictating<br>
> >>>>> design intent.<br>
> >>>>><br>
> >>>>> The idea of adding it as a *recommendation* would merely be to raise<br>
> >>>>> awareness of an area that Theme developers may not previously have<br>
> >>>>> considered styling.<br>
> >>>>><br>
> >>>> Sounds good to me.<br>
> >>>><br>
> >>>> Ian Stewart<br>
> >>>><br>
> >>>> ______________________________**_________________<br>
> >>>> theme-reviewers mailing list<br>
> >>>> theme-reviewers@lists.**<a href="http://wordpress.org" target="_blank">wordpress.org</a><<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><<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>><br>
> >>>><br>
> >>>><br>
> >>>> ______________________________**_________________<br>
> >>> theme-reviewers mailing list<br>
> >>> theme-reviewers@lists.**<a href="http://wordpress.org" target="_blank">wordpress.org</a><<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><<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>><br>
> >>><br>
> >><br>
> >><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>
> >><br>
> ><br>
><br>
> --<br>
> Sent from my mobile device<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>
> _______________________________________________<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>
_______________________________________________<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>