seandowney/bonfire_navigationmodule — pull request #4 by svizion (2011-10-22 11:40:11+00:00)
Title: Minor changes to wrap li's in span, and change the name of the current class

This is the changes I made for the site I built using your module, It has 2 extra param's in your $attributes setting. They do not have to be set and it works the same as it currently does.

If you set

$attributes['wrap'] = true;

Then Your Output will be

Title

I also ran into a issue with the Bootstrap UI using active as there current class name so I added a option to change the current location's class name

attributes['active'] = 'active';

Will change the class to active, if not set it will stay the same as it is now ( current ).

I personally found these changes needed for the design I was working on and other people might find them useful also.

Thanks for making this module it was quite helpful in learning some of Bonfire's admin settings and stuff.

Shawn