<div dir="ltr"><div>The theme should make use of the core functionality ie front-page, rather than having an option to enable frontpage.<br><br>Other than this rest is fine.<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 28, 2014 at 1:41 AM, Hardeep Asrani <span dir="ltr"><<a href="mailto:hardeepasrani@gmail.com" target="_blank">hardeepasrani@gmail.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>I'm reviewing a theme which has an option on its Theme Options pages to "Enable Frontpage" which then uses front-page.php as the homepage of the site.<br><br>Do you think a theme can have this option it it, especially when this disobeys the option to choose Front/Blog page in the Reading settings?<br><br>I'm pretty sure it should be removed, but still wanted to get your opinion :)<br><br>Also that theme is using <b>WEBRITI_TEMPLATE_DIR_URI</b> instead of <b>get_template_directory_uri</b> after defining it in the following lines:<br><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">define('WEBRITI_TEMPLATE_DIR_URI',get_template_directory_uri());<br></blockquote><br></div>Is it okay?<span class="HOEnZb"><font color="#888888"><br><div><div>-- <br>Regards,<br>Hardeep Asrani<br><br></div></div></font></span></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>

</blockquote></div> <br clear="all"> --  GNU/LINUX User <a href="http://fsf.org" target="_blank">Free software</a> is the matter of liberty not price.

</b>

</div>