Why the overloads with class parameter in Html.Menu?

Sep 24, 2009 at 6:14 AM

<%=Html.Menu("someprovider", "someclass", "", new { id = "testMenu"})%>

Why have you provided this overload?  This can be easily accomplished by:

<%=Html.Menu("someprovider", "", new { id = "testMenu" @class="someclass"})%>

This would take down the number of overloads considerably.  Note you have specific code to make sure it is not set like I have shown above.  Just allow it and remove that code!

Also, there are other overloads needed.  You provide a 'startingNodeLevel' - really, we need the menu to be able to start from a given URL many times the CURRENT url.

Thoughts?  If you agree - I will make all these changes and enhancments for you - just let me know.


Sep 24, 2009 at 6:50 AM

That's valuable input! If you want, you can make the changes. I've added you as a contributor to this project so you can check-in directly. Make sure to always create a work item on the issue tracker when doing a check-in.

Really appreciate your contributions!

Sep 28, 2009 at 7:42 PM
This discussion has been copied to a work item. Click here to go to the work item and continue the discussion.