RWToolbarList String

Hello Dan & Ben,

I would like to present a limitation/issue and then present a hopeful solution that I would ask that you consider.
It has to do with the Theme.plist, particularly the toolbar keys/strings.

Currently there are these keys:

  • RWToolbarList
  • RWToolbarItemNormalParent
  • RWToolbarItemNormal
  • RWToolbarItemCurrentParent
  • RWToolbarItemCurrentAncestor
  • RWToolbarItemCurrent

and then there are these macros that build out the toolbar:

  • %items%
  • %subitems%

These are great but it creates a problem with responsive designs because lets say that you have set up your theme to create the list like this:
<ul class=“nav navbar-nav”>
<li><a href="#">item</a></li>
</ul>

then if you have sub menu items the child ul automatically recieves the class names of the parent ul and creates a styling nightmare:

<ul class=“nav navbar-nav”>
<li><a href="#">item</a>
<ul class=“nav navbar-nav”>
<li><a href="#">sub-item</a></l>
</ul>
</li>
</ul>

What this means is that we have to rely on js/jquery to look for the child ul and strip the class names of the parent ul and often add new class names to the child ul so that we can apply the necessary styling.

What I would LOVE to see implemented is one of the following:

  1. Add a new key/string to the plist RWToolbarListChild or RWToolbarListSub
    This key would target the %subitems% macro. or
  2. Apply any class names of the RWToolbarList to ONLY the parent ul

#1 would be my preference.

Thanks
Gary