# Creating a special UL LI menu and having issues with subpage **URL:** \ **Category:** Rails\ **Created:** [December 25, 2009, 3:57pm UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666 "2009-12-25T15:57:21Z")\ **Posts on this page:** 5\ **Page:** 1
**Author:** ![elricstorm](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/e/da6949/32.png) [@elricstorm](https://www.ruby-forum.com/u/elricstorm)\ **Post date:** [December 25, 2009, 3:57pm UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666/1 "2009-12-25T15:57:21Z")
I’m having a little trouble with trying to get a UL menu working with rails. It’s working somewhat, but not the way it needs to. I’ll explain after I post the code. First, I’ve created a page administration system similar to what was created in the screencasts for learningrails: [http://www.buildingwebapps.com/learningrails](http://www.buildingwebapps.com/learningrails) What I like about the page administration system is having the ability to create pages on the fly without having to use a hard redirect to a controller. This is especially good for pages which are not static and always being changed. However, they use a standard menu system and do not showcase a dropdown menu system. I wanted to create a dropdown menu system instead. The following code is in the application controller: define get\_pages\_for\_tabs @tabs = Page.find\_main @subs = Page.find\_sub end In the page model the code does the following: # this method defines the tab views for all tabs def self.find\_main Page.find(:all, :conditions =\> [‘parent\_id IS NULL’], :order =\> ‘position’) end # this method defines the subpage tab views for all tabs def self.find\_sub Page.find(:all, :conditions =\> [‘parent\_id IS NOT NULL’], :order =\> ‘position’) end What it’s going to do simply put, is find all pages that do not have a parent\_id set to them, which infers that those pages are -parent- pages. It sets those parent pages to @tabs. It then finds all pages that do have parent\_ids, which infers that those pages are -subpages- of the parent. It sets those pages to @subs, ordering them by position numbers set. The following code below is listed in the application.html.erb file in my view layouts. \<% @tabs.each do |page| -%\> \<% if page.redirect? %\> - \<%= link\_to page.navlabel, {:action =\> page.action\_name, :controller =\> page.controller\_name, :name =\> page.name} -%\> \<% else %\> - \<%= link\_to page.navlabel, view\_page\_path(page.name) %\> \<% @subs.each do |subpage| -%\> \<% if page.id == subpage.parent\_id %\> - \<%= link\_to subpage.navlabel, view\_page\_path(subpage.name) %\> \<% end %\> \<% end %\> \<% end %\> \<% end -%\> So, looking at the code above, here’s what will happen: If a page has a redirect box checked, it uses an MVC. If it doesn’t, it uses a non-static database driven page instead. The page model houses the following fields: id, name, title, body, parent\_id, navlabel, position, redirect, action\_name, controller\_name If a page is going to be a sub-nav from a parent page menu item, I’ll simply define the parent\_id for that subpage which will match the parent.id. The only problem with the code above is that it only displays one sub-menu for each parent menu item. If I have a parent page id of say 4 and I create three subpages that have parent\_id = 4, it should look at the following code: \<% if page.id == subpage.parent\_id %\> … and showcase it. It does so for only one subpage though. What am I doing wrong based on the code shown? If you have any further questions about this, please let me know. Hopefully I provided all the information needed. By the way, Merry Christmas everyone. 🙂 ---
**Author:** ![elricstorm](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/e/da6949/32.png) [@elricstorm](https://www.ruby-forum.com/u/elricstorm)\ **Post date:** [December 26, 2009, 4:50am UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666/2 "2009-12-26T04:50:11Z")
Offering a visual example of what happens currently: –Parent Menu Button I ----Fourth Sub Menu Button I –Parent Menu Button II ----Fourth Sub Menu Button II when what should happen is: –Parent Menu Button I ----First Sub Menu Button I ----Second Sub Menu Button I ----Third Sub Menu Button I ----Fourth Sub Menu Button I –Parent Menu Button II ----First Sub Menu Button II ----Second Sub Menu Button II ----Third Sub Menu Button II ----Fourth Sub Menu Button II So, it’s creating the menu buttons using UL Li styles but it’s only placing the last instance of each array into the menu, versus adding all of them. When outputting this using puts in a rails console, it outputs everything properly. So, I’m not sure what’s wrong with my code. Any help would be appreciated. ---
**Author:** ![elricstorm](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/e/da6949/32.png) [@elricstorm](https://www.ruby-forum.com/u/elricstorm)\ **Post date:** [December 26, 2009, 7:31pm UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666/3 "2009-12-26T19:31:21Z")
On Dec 25, 9:57 am, Alpha B. [removed\_email\_address@domain.invalid](mailto:removed_email_address@domain.invalid) wrote: > to create pages on the fly without having to use a hard redirect to a > end First off, you’d be better off actually setting up some real associations. Such as: class Page \< ActiveRecord::Base belongs\_to :parent, :class\_name =\> ‘Page’ has\_many :sub\_pages, :class\_name =\> ‘Page’, :foreign\_key =\> ‘parent\_id’, :order =\> ‘position’ named\_scope :main\_pages, :conditions =\> ‘pages.parent\_id IS NULL’, :order =\> ‘pages.position’, :include =\> :sub\_pages end The controller code basically disappears; you can stash the results of Page.main\_pages in an instance variable if you’re hell-bent on avoiding model calls in the view. Here’s a cleaned-up version of the view code: > > It does so for only one subpage though. > > What am I doing wrong based on the code shown? You didn’t mention if the actual source from your original view includes all the elements; I’d hazard that the CSS for the menus might be dropping them all in a stack, which would produce the “only show the last one” behavior you’ve described. Hope this helps! –Matt J. – You received this message because you are subscribed to the Google Groups “Ruby on Rails: Talk” group. To post to this group, send email to removed\_email\_address@domain.invalid. To unsubscribe from this group, send email to removed\_email\_address@domain.invalid. For more options, visit this group at [http://groups.google.com/group/rubyonrails-talk?hl=en](http://groups.google.com/group/rubyonrails-talk?hl=en). ---
**Author:** ![elricstorm](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/e/da6949/32.png) [@elricstorm](https://www.ruby-forum.com/u/elricstorm)\ **Post date:** [December 27, 2009, 4:54am UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666/4 "2009-12-27T04:54:47Z")
Hi Matt, Thanks a bunch for the helpful code. I didn’t post my model associations but I did have all of the associations in place, like you suggested, with the exception of placing in a built-in order for position. So I added that and changed sub\_pages to subpages throughout the code and also added in the scope which makes a lot of sense. After testing the code, everything works as intended now. I’m going to spend some time mulling over what it is I just did thanks to your help, but I wanted to reply and let you know that it’s all working fine now with your code changes and suggestions in place. I appreciate it a great deal. Take care mate. ---
**Author:** ![elricstorm](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/e/da6949/32.png) [@elricstorm](https://www.ruby-forum.com/u/elricstorm)\ **Post date:** [December 28, 2009, 1:03am UTC](https://www.ruby-forum.com/t/creating-a-special-ul-li-menu-and-having-issues-with-subpage/180666/5 "2009-12-28T01:03:58Z")
Hi Matt and Everyone, As always, I like to follow-up with detailed solutions and feedback on end topics I create. I’m adding a link to gist which houses all of the changes and what works perfectly now: > > > There are more than three files. show original In the application.html.erb layout, I don’t particularly like having too much code, even code that pulls from the model. I like to move that into a helper file and since the menu resides within a table, I placed the real code in the table helper file. In the application.html.erb layout, all you have to use is a simple call to the helper method of menu. You’ll see that I start with the call for @tabs in the application\_controller. I mainly due this because I have administrative tabs that I don’t want people to see, so I created a before\_filter that looks for admin sessions and if so, admin tabs are displayed and if not, the public tabs are shown. In the page.rb model, you’ll see that I am now using the scope you created, but I’m also using another scope that applies to admin tabs. Finally, in the table\_helper.rb file, you’ll see the code which controls two situations (those where true redirects are being used on the menu or those where virtual menu redirects are being used). However, it applies situational responses for both parent and sub-menu tabs. I’m sure you’ll understand it once you see it. And, I like to share code with folks that might have a similar issue they need solved. As always, if you see any issues or you have ideas to clean up the resolved code, please post your response so others browsing later on, can find a cleaned up version of this topic. Take care all.