Page 1 of 1

Collapse global navigation with other theme options?

Posted: Thu Feb 12, 2015 10:49 pm
by Don Chambers
Hey, we have this in bulletproof:

Code: Select all

$template_config_groups = array(
    THEME_COLORSET  => array('colorset', 'layouttype', 'jscolumns'),
    THEME_HEADER    => array('custheader', 'headerimage', 'headertype')
);
So, the theme options for colorset, layouttype, and jscolumns all collapse under the heading defined by THEME_COLORSET.

Is it possible to collapse global navigation under a heading as well?

Re: Collapse global navigation with other theme options?

Posted: Fri Feb 13, 2015 9:23 am
by garvinhicking
Hm, problaby by using the exact configuration key name of the global navigation. I'd have to check...

Re: Collapse global navigation with other theme options?

Posted: Fri Feb 13, 2015 11:18 pm
by Don Chambers
I know I am close with this:

Code: Select all

$navlinks_collapse = array();
array_push($navlinks_collapse, 'use_corenav', 'amount');
for ($i = 0; $i < $template_loaded_config['amount']; $i++) {
	array_push($navlinks_collapse, 'navlink' . $i . 'text' ,'navlink' . $i . 'url');
}

$template_config_groups = array(
    THEME_NAVIGATION    => $navlinks_collapse
);
I have the "use_corenav", and "amount" collapsed, but I'm not getting navlink0text, navlink0url, etc into $navlinks_collapse... which suggests something is wrong with my loop... can I not use $template_loaded_config['amount'] when navigation is global? This all works just fine if the links are defined within the theme (not global).

Re: Collapse global navigation with other theme options?

Posted: Sun Feb 15, 2015 7:19 pm
by Don Chambers
$template_loaded_config['amount'] simply doesn't work here. Anyone have a better suggestion?

Re: Collapse global navigation with other theme options?

Posted: Mon Feb 16, 2015 10:28 am
by garvinhicking
If you can create a github issue and assign it to me, I'll try to inspect this in a few days so I don't forget it?

Re: Collapse global navigation with other theme options?

Posted: Mon Feb 16, 2015 10:37 pm
by Don Chambers
Done.