Thanks for theme - The theme works nicely except for this:

The class of selected(active) menu li is not being set to active
So this CSS rule is not being called: .navbar-nav > .active
This stops the selected menu item from getting the yellow rounded box set around it

[Separated the two issues into their own issue]

Comments

wavesailor created an issue. See original summary.

wavesailor’s picture

blamege1’s picture

any progress on this?

blamege1’s picture

Just a heads up to anyone else who's having this issue.

I was planning on adding some custom stuff to the head, so I copied the html.tpl.php file from the /modules/systems folder into my theme folder at sites/all/themes/bootstrap_agency/templates. Suddenly the active buttons seemed to work. Not sure why, but they did.

wavesailor’s picture

@blamege1 that does seem to work - I don't know enough to figure out why.

My only concern is I think we should be copying the html.tpl.php file from the themes/bootstrap/templates/system/ and not the drupal core version.
Do we lose any bootstrap functionality by using the drupal core version?

Any ideas on how to solve / stop the background image used on the <header> from being cropped? It leaves a horrible blank space at the bottom.

blamege1’s picture

@wavesailor, I'm going to guess you meant "that doesn't seem to work" above?

My only concern is I think we should be copying the html.tpl.php file from the themes/bootstrap/templates/system/ and not the drupal core version.
Do we lose any bootstrap functionality by using the drupal core version?

I can only tell you what I did. When comparing the two files themes/bootstrap/templates/system/html.tpl.php and modules/system/html.tpl.php, there's not much difference that I can see.

namwebs’s picture

#4 didn't work for my site and the selected menu item doesn't show as active. Has anybody fixed this another way?

wavesailor’s picture

Title: Selected Main Menu item class not being set active and header background image cropped » Selected Main Menu item class not being set active
Issue summary: View changes

Separated the two issues into their own issue.

wavesailor’s picture

@blamege1 Your solutions works but causes the site not to display properly on mobile devices.