I would concur with your comments.<div><br></div><div>There's absolutely no reason to dump dummy links in a sidebar like that. Use a call to <b>wp_list_bookmarks()</b> or <b>the_widget( 'WP_Widget_Links' )</b>, at the very least.</div>
<div><br></div><div>Chip<br><br><div class="gmail_quote">On Mon, Apr 18, 2011 at 9:07 PM, Philip Walton <span dir="ltr"><<a href="mailto:philip@philipwalton.com">philip@philipwalton.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I've been reviewing several versions of the same theme and the author has chosen to include a menu full of placeholder links with href="#"<br>
<br>
<a href="http://themes.trac.wordpress.org/ticket/3556" target="_blank">http://themes.trac.wordpress.org/ticket/3556</a><br>
<br>
The last link in the list is titled "How to edit this menu?" and points to: <a href="http://en.support.wordpress.com/menus/" target="_blank">http://en.support.wordpress.com/menus/</a>. It seems he's just using all those empty links to demonstrate a look, and he assumes its uselessness will be a more effective way of getting people to actually customize the menu.<br>
<br>
The theme had several errors, so I did not approve it, and in the comments I suggested that he not list empty links because I thought it would be confusing to people (and it just seems wrong). He respectfully disagreed with me and uploaded a new version with the same empty links.<br>
<br>
Anyway, I wanted to run this by you all to see if you think this type of thing should be acceptable in a theme.<br>
<br>
_______________________________________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">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></div>