<div dir="ltr">Hi Bryan,<div><br></div><div>I'll be happy to take a look at this. Can you comment in-ticket first, so that I can follow up on your responses to the original reviewer?</div><div><br></div><div>Thanks,</div>
<div><br></div><div>Chip</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 19, 2013 at 9:19 PM, Bryan Hadaway <span dir="ltr"><<a href="mailto:bhadaway@gmail.com" target="_blank">bhadaway@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><a href="http://themes.trac.wordpress.org/ticket/11498" target="_blank">http://themes.trac.wordpress.org/ticket/11498</a><br>
<br>Theme was not approved over several misunderstandings of recommended vs required. A lot of this stuff we've already gone over.<br>
<br>Issues for not-approved:<br><br>- <b>Licensing</b>. All 3rd party/licensed scripts already do contain the proper license info. Do I still need to add a license.txt in addition?<br><br>- <b>Inline Styles</b>. Recommended not to use, but not required. I recall this discussion.<br>
<br>- <b>Site Title</b>. The site title does show in plain text if no logo/header is set. I have no problem with this in my testing, works fine.<br><br>So, the only thing I think might need to be added is a license.txt, only if required in addition to the scripts themselves already containing the license info. Which is fine, but can I get an admin to approve this theme or take it over and I'll upload the new version based on your instruction.<br>
<br>Thanks<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></blockquote></div><br></div>