<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><div class="gmail_quote">On Sat, Jul 16, 2011 at 8:43 AM, Chip Bennett <span dir="ltr"><<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I'll link the existing ticket when I'm back at my laptop; it's been wontfixed.<br>
<br>
Chip<br>
<div><div></div><div class="h5"><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>
</div></div>>>>> 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>
<div class="im">>>><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>
</div><font color="#888888">--<br>
Sent from my mobile device<br>
</font><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>