<div dir="ltr">Chip, clarified that generic logo's are fine, they just can't be theme specific.<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Feb 17, 2013 at 1:04 PM, Greg Priday <span dir="ltr"><<a href="mailto:greg@siteorigin.com" target="_blank">greg@siteorigin.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I did a quick check of <a href="http://themes.trac.wordpress.org/ticket/11081" target="_blank">http://themes.trac.wordpress.org/ticket/11081</a>.<br>
There are several problems that should have prevented its approval.<br>
Incorrectly prefixed functions, error messages with WP_DEBUG=true and<br>
a few JS and image license issues. I'll add my review to the ticket.<br>
<br>
Also, how should themes handle default logos? I feel they should be<br>
treated the same as favicons, which are required to be disabled by<br>
default. Themes should default to a text only, site_title logo.<br>
<div><div class="h5"><br>
<br>
<br>
On Sun, Feb 17, 2013 at 8:56 AM, Srikanth Koneru <<a href="mailto:tskk79@gmail.com">tskk79@gmail.com</a>> wrote:<br>
> As Bryan explained, flexibility is fine.<br>
><br>
><br>
> On Sun, Feb 17, 2013 at 12:15 PM, Abhik Biswas <<a href="mailto:abhik@itsabhik.com">abhik@itsabhik.com</a>> wrote:<br>
>><br>
>> Why not making a system that makes only the top theme in the queue<br>
>> eligible to be assigned and reviewed?<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>
> 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></div>--<br>
I make free WordPress themes<br>
<a href="http://siteorigin.com" target="_blank">http://siteorigin.com</a><br>
<a href="http://facebook.com/SiteOrigin" target="_blank">http://facebook.com/SiteOrigin</a><br>
<a href="http://twitter.com/SiteOrigin" target="_blank">http://twitter.com/SiteOrigin</a><br>
<div class="HOEnZb"><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>