<div dir="ltr">hello @devin I respect your code but i think there should a way to <span style="font-family:arial,sans-serif;font-size:13px">flexibility and usability may be some of people like me getting this problem with latest version while uploading theme over WP.org. You can add a filter that not deny theme from uploading on WP so both problem will resolve and it's up to you.</span><div>

<span style="font-family:arial,sans-serif;font-size:13px"><br></span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Regards</span></div><div><span style="font-family:arial,sans-serif;font-size:13px">Sandeep</span></div>

</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 8, 2014 at 9:13 PM, Devin Price <span dir="ltr"><<a href="mailto:devin@wptheming.com" target="_blank">devin@wptheming.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I wanted to respond to @Sandeep's issue with the Options Framework.  I recently introduced a filter to allow a little more flexibility with the options pages.  This allows people to add a top level page, but the default is to put it under the "Appearance Menu" (<a href="https://github.com/devinsays/options-framework-theme/commit/82d68aa9000f291d83c03f5a610e2037dd21d6bf" target="_blank">https://github.com/devinsays/<u></u>options-framework-theme/<u></u>commit/<u></u>82d68aa9000f291d83c03f5a610e20<u></u>37dd21d6bf</a>).<br>


<br>
Obviously for .org theme review, you would need to use the default and and load it under the appearance menu.  I guess there's a couple options:<br>
<br>
a)  I could remove the filter and flexibility for the library<br>
b)  The theme check could be altered to still flag for add_menu_page and list the requirement of loading options pages under the appearance menu, but not deny the upload.<br>
<br>
Thoughts?<br>
<br>
Devin Price<br>
<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</blockquote></div><br></div>