<div dir="ltr">Thanks! :)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 23 March 2014 22:58, Otto <span dir="ltr"><<a href="mailto:otto@ottodestruct.com" target="_blank">otto@ottodestruct.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Sun, Mar 23, 2014 at 2:06 PM, Ulrich Pogson <span dir="ltr"><<a href="mailto:grapplerulrich@gmail.com" target="_blank">grapplerulrich@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Redux Framework uses add_submenu_page() instead of add_theme_page(). Can we make an exception here if the the added menu item is under "Appearance"?</div>



</blockquote><div><br></div></div><div>No, because that is what add_theme_page is for.</div><div><br></div><div>The add_theme_page function is just a wrapper for add_submenu_page that automatically puts in the "themes.php" name for the first parameter. </div>



<div><br></div><div>So, if at some future point, the files get a renaming or the menu system gets a big heavy reorganization, then themes using add_theme_page would not need any changes to continue working properly.</div>



<div><br></div><div>When high level wrappers exist for this sort of thing, then plugin/theme code should use them for maximum future compatibility.</div><span class="HOEnZb"><font color="#888888"><div><br clear="all"><div>

-Otto</div></div><div><br></div><div><br>
</div>
</font></span></div></div></div>
<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>