<div dir="ltr">While I would like to see us get there, we don't currently *require* that the settings page output its markup using settings_fields(). As long as the markup is valid, and untrusted data are properly escaped, "hard-coded" markup in the settings page is acceptable.<div>
<br></div><div>(I don't foresee that changing unless/until the Settings API is changed/improved to include core markup for various data/field types.)</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Sat, Jan 18, 2014 at 9:20 AM, ethan anwar <span dir="ltr"><<a href="mailto:ethananwar@yahoo.com" target="_blank">ethananwar@yahoo.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div style="font-size:12pt;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><div><span>Hi and sorry to bother you all again!</span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">
<br><span></span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><span>I'm reviewing another theme ( <a href="https://themes.trac.wordpress.org/ticket/15946" target="_blank">https://themes.trac.wordpress.org/ticket/15946</a> ) and I have noticed what I consider blurry use of the Theme Settings API. <br>
</span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><span><br></span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">
<span>Basically the theme's developer uses all the right functions, registers the main settings array via register_setting() but doesn't add any settings via add_settings_field(). Then, inside the function for the actual options page he uses settings_fields() then<br>
do_settings_sections() which probably don't return anything. After this, HTML follows with the actual code for the settings.</span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">
<br><span></span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><span>What's your opinion on this? Shouldn't he use add_settings_section() and add_settings_field() if he's going with do_settings_sections() and settings_fields() ?</span></div>
<div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif"><br><span></span></div><div style="font-style:normal;font-size:16px;background-color:transparent;font-family:HelveticaNeue,Helvetica Neue,Helvetica,Arial,Lucida Grande,sans-serif">
<span>As always thanks a lot for your time guys!<br></span></div><div style="display:block">Kay<br>  </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>