#4357 Build top nav using main_menu() instead of sitemap() methods

v1.0.0
closed
General
nobody
2015-08-20
2012-06-12
No

The top nav of a project is built using project.sitemap() which calls sitemap() for all the tools, which frequently call their sidebar_menu methods. Those may do a lot of unnecessary extra work, since all that is needed is the top menu.

Each app should also define a main_menu() method. Let's make sure all of those are defined properly in our apps, and then make the top nav render only use those. Consider if Project.sitemap is used elsewhere - maybe it can be simplified or maybe a new method needs to be created. Also see how Project.menus is used.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2012-09-11

    This goes hand-in-hand with [#4718]

     

    Related

    Tickets: #4718

  • Dave Brondsema

    Dave Brondsema - 2012-09-11
    • status: open --> in-progress
    • assigned_to: Dave Brondsema
    • milestone: forge-backlog --> forge-sep-21
     
  • Dave Brondsema

    Dave Brondsema - 2012-09-17
    • status: in-progress --> closed
     
  • Dave Brondsema

    Dave Brondsema - 2012-09-17

    Fixed in [#4718] which is now closed

     

    Related

    Tickets: #4718


Log in to post a comment.