- changed status to resolved
Make Number of Menus Selectable
Currently there are 10 hard coded menu fields in the "Header Menus" section. This should be changed to use the process found under "Alert Box" (field alertcount). This adds a select box which allows users to define the number of alerts they wish to use.
This relies on a constant on line 27 of lib.php
define('THEME_ADAPTABLE_DEFAULT_ALERTCOUNT', '1');
Code in settings.php begins on line 2761 where alertcount is defined. On line 2776 we see this being looped in the admin settings:
for($alertindex=1;$alertindex <= $alertcount;$alertindex++){
On the front end code to render menus can be found on line 697 where the function public "function get_top_menus()" is declared.
on line 709 of renderers.php we currently loop the hard coded admin settings:
for ($i = 1; $i < 11; $i++){
As long as the naming of admin fields (within the loop in settings.php) is consistent, the rest of the code in renderers.php should work as it does now.
Comments (1)
-
reporter - Log in to comment
Fix
#218- make number of top menus definable by administrator.Also moved menu labels / titles out of language pack into a setting to make it clearer to users how to set.
→ <<cset 773372990a70>>