Problem/Motivation
Responsive Dropdown Menus blocks always show 3 tiers of a menu. We had a business requirement to only show 2 tiers of the menu.
It should also be mentioned that the current use of menu_tree_all_data() passes the default (3) as the second argument, when $max_depth is in fact the third parameter (see function docs). That will also be fixed.
Proposed resolution
Add a configuration option (that still defaults to 3) in the Responsive Dropdown Menus blocks that allows site builders to choose between showing 2 or 3 tiers.
Remaining tasks
Review/test the attached patch and, if it is deemed acceptable, commit it! :)
User interface changes
Responsive Dropdown Menus will now have a configuration option unique to the module.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | responsive_dropdown_menus-depth-config-2415377-1.patch | 2.41 KB | guschilds |
Comments
Comment #1
guschilds commentedAttached the patch to review/test. Should apply to the current state of the 7.x-1.x branch. Thanks!
Comment #2
mglamanAwesome enhancement, thanks!