Hi there,<div><br>I&#39;m reviewing the inLine theme from the queue (number 4584, link - <a href="http://themes.trac.wordpress.org/ticket/4584">http://themes.trac.wordpress.org/ticket/4584</a> ). I&#39;m pretty positive about the code quality of the theme, the admin settings and the behavior based on the demo content. Theme checker plugins throw no exceptions, post formats are defined appropriately (except aside which shows title here) and almost everything looks great to me. The theme is applying additional hooks and the code is module-based and commented as well.</div>
<div><br></div><div>My only serious remark is that by default the theme top menu shows &quot;Home&quot; only. Not a standard menu, not the fallback page menu that WordPress applies by default. Just the Home link.</div><div>
<br>After code inspection I found that code snippet which expects a primary location for custom menu even if none is defined:</div><div><br></div><div><br></div><div><div>function inline_do_header_nav() {</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>// Create the primary nav menu</div>
<div><span class="Apple-tab-span" style="white-space:pre">        </span>$inline_primary_nav = wp_nav_menu( array( &#39;theme_location&#39; =&gt; &#39;primary&#39;, &#39;sort_column&#39; =&gt; &#39;menu_order&#39;, &#39;container_id&#39; =&gt; &#39;primary&#39;, &#39;container_class&#39; =&gt; &#39;menu-header&#39;, &#39;echo&#39; =&gt; &#39;false&#39;, &#39;fallback_cb&#39; =&gt; &#39;inline_fallback_primary_nav&#39; ) );</div>
<div><span class="Apple-tab-span" style="white-space:pre">                                                </span></div><div><span class="Apple-tab-span" style="white-space:pre">        </span>// Display the primary nav menu only if it is set</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>if ( $inline_primary_nav ) {</div>
<div><span class="Apple-tab-span" style="white-space:pre">                </span>echo $inline_primary_nav;</div><div><span class="Apple-tab-span" style="white-space:pre">        </span>}<span class="Apple-tab-span" style="white-space:pre">        </span></div>
<div>}</div><div><br></div><div><br></div><div>I assume this is against the standard rules - or maybe it isn&#39;t? </div><div><br>If this expectation is allowed, I would suggest &#39;approved&#39; status for the theme but I find it inappropriate for the fallback function to list &#39;Home&#39; only when I have 12 pages on my demo site. </div>
<div><br>Thanks for the help,</div><br>Mario Peshev<br>freelance software developer/trainer<br><a href="http://www.linkedin.com/in/mpeshev">http://www.linkedin.com/in/mpeshev</a><br><a href="http://peshev.net/blog">http://peshev.net/blog</a><br>
<br>
</div>