<br>At this point, what&#39;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">&lt;<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I&#39;ll link the existing ticket when I&#39;m back at my laptop; it&#39;s been wontfixed.<br>
<br>
Chip<br>
<div><div></div><div class="h5"><br>
On 7/16/11, Edward Caissie &lt;<a href="mailto:edward.caissie@gmail.com">edward.caissie@gmail.com</a>&gt; wrote:<br>
&gt; I&#39;m reading &quot;needs patch&quot; from this discussion to &quot;correct&quot; the method core<br>
&gt; is currently using.<br>
&gt;<br>
&gt; As a recommendation, I&#39;m all for it ... if we can have core &quot;fixed&quot; to a<br>
&gt; more &quot;correct&quot; method or default style then I would entertain the<br>
&gt; possibility of pushing the recommendation to a requirement as we have done<br>
&gt; with other guidelines.<br>
&gt;<br>
&gt;<br>
&gt; Cais.<br>
&gt;<br>
&gt;<br>
&gt; On Fri, Jul 15, 2011 at 7:56 PM, Chip Bennett &lt;<a href="mailto:chip@chipbennett.net">chip@chipbennett.net</a>&gt; wrote:<br>
&gt;<br>
&gt;&gt; +1 to this:<br>
&gt;&gt;<br>
&gt;&gt; *With that said, WP&#39;s handling of this is crap.*<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; And therein lies the problem with even considering making it a<br>
&gt;&gt; requirement.<br>
&gt;&gt; The underlying implementation (inline styles by default, and force Theme<br>
&gt;&gt; developers to remove them via filter) impedes Theme developers - but I&#39;ve<br>
&gt;&gt; tried to fight that fight, and lost. So, as long as WordPress core<br>
&gt;&gt; continues<br>
&gt;&gt; NOT to do it right, I think it would be an unreasonable burden on Theme<br>
&gt;&gt; developers to *require* them to support more than the default number of<br>
&gt;&gt; columns.<br>
&gt;&gt;<br>
&gt;&gt; Chip<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Sat, Jul 16, 2011 at 6:33 PM, Justin Tadlock<br>
&gt;&gt; &lt;<a href="mailto:justin@justintadlock.com">justin@justintadlock.com</a>&gt;wrote:<br>
&gt;&gt;<br>
&gt;&gt;&gt; Part of me thinks it should be a requirement.  WordPress already handles<br>
&gt;&gt;&gt; styles for 1 - 10 columns (and more) by default.  If a theme is<br>
&gt;&gt;&gt; overwriting<br>
&gt;&gt;&gt; this with custom styles or removing the inline &lt;style&gt; WP adds, they need<br>
&gt;&gt;&gt; to<br>
&gt;&gt;&gt; handle this on their own.  That&#39;s where the biggest problem is -- themes<br>
&gt;&gt;&gt; overwriting this but not handling anything other than 3 columns.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; With that said, WP&#39;s handling of this is crap.  I&#39;ve never designed a<br>
&gt;&gt;&gt; theme where it doesn&#39;t break with the default WP styles.  Of course, I<br>
&gt;&gt;&gt; always roll my own handling of this anyway.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On 7/15/2011 5:39 PM, Ian Stewart wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; On 2011-07-15, at 5:36 PM, Chip Bennett wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;  Personally, I wasn&#39;t even thinking along those lines. Making something<br>
&gt;&gt;&gt;&gt;&gt; like that a *requirement* seems, IMHO, to cross the line into dictating<br>
&gt;&gt;&gt;&gt;&gt; design intent.<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; The idea of adding it as a *recommendation* would merely be to raise<br>
&gt;&gt;&gt;&gt;&gt; awareness of an area that Theme developers may not previously have<br>
&gt;&gt;&gt;&gt;&gt; considered styling.<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Sounds good to me.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Ian Stewart<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; ______________________________**_________________<br>
&gt;&gt;&gt;&gt; theme-reviewers mailing list<br>
</div></div>&gt;&gt;&gt;&gt; theme-reviewers@lists.**<a href="http://wordpress.org" target="_blank">wordpress.org</a>&lt;<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>&gt;<br>

&gt;&gt;&gt;&gt; <a href="http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers" target="_blank">http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers</a>&lt;<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>&gt;<br>

&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;  ______________________________**_________________<br>
&gt;&gt;&gt; theme-reviewers mailing list<br>
&gt;&gt;&gt; theme-reviewers@lists.**<a href="http://wordpress.org" target="_blank">wordpress.org</a>&lt;<a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a>&gt;<br>
&gt;&gt;&gt; <a href="http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers" target="_blank">http://lists.wordpress.org/**mailman/listinfo/theme-**reviewers</a>&lt;<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a>&gt;<br>

<div class="im">&gt;&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; _______________________________________________<br>
&gt;&gt; theme-reviewers mailing list<br>
&gt;&gt; <a href="mailto:theme-reviewers@lists.wordpress.org">theme-reviewers@lists.wordpress.org</a><br>
&gt;&gt; <a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/mailman/listinfo/theme-reviewers</a><br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;<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>