Make Number of Menus Selectable

Issue #218 resolved
Jeremy Hopkins created an issue

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)

  1. Jeremy Hopkins reporter

    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>>

  2. Log in to comment