Sorry, the appropriate ID / link is 4695 (newer submission) - <a href="http://themes.trac.wordpress.org/ticket/4695">http://themes.trac.wordpress.org/ticket/4695</a><br clear="all"><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>
<br><br><div class="gmail_quote">On Mon, Aug 1, 2011 at 3:56 PM, Mario Peshev <span dir="ltr">&lt;<a href="mailto:mario@peshev.net">mario@peshev.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
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" target="_blank">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 style="white-space:pre-wrap">        </span>// Create the primary nav menu</div>

<div><span style="white-space:pre-wrap">        </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 style="white-space:pre-wrap">                                                </span></div><div><span style="white-space:pre-wrap">        </span>// Display the primary nav menu only if it is set</div><div><span style="white-space:pre-wrap">        </span>if ( $inline_primary_nav ) {</div>

<div><span style="white-space:pre-wrap">                </span>echo $inline_primary_nav;</div><div><span style="white-space:pre-wrap">        </span>}<span style="white-space:pre-wrap">        </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" target="_blank">http://www.linkedin.com/in/mpeshev</a><br><a href="http://peshev.net/blog" target="_blank">http://peshev.net/blog</a><br>

<br>
</div>
</blockquote></div><br>