Works as designed for screen widths less than 980px.

Anything wider than 979px the menu is displayed even if the '.in' class is not assigned.

I've hit my head against the wall too much on this one and could use a second set of eyes. Please.

Comments

frankbaele’s picture

Status: Active » Postponed (maintainer needs more info)

could you give some more info so we can reproduce your situation and screenshots

joshuautley’s picture

StatusFileSize
new5.03 KB
new18.63 KB

Screen grabs attached. Thank you for the quick response.

joshuautley’s picture

Frank,

The only thing I could come up with to fix this issue was to edit the bootstrap-responsive.css

changed the following

line 935
@media (max-width: 1154px) { /* was 979px */

then on line 1083
@media (min-width: 1155px) { /* was 980px */

I've tried several methods in my own style sheet before resorting to changing these values in yours. I understand why you're using 980, it's the standard/default for mobile and works with a 960 grid layout, but we're not using a 960 grid layout.

If you can suggest a method for overriding this in my style sheet it would be greatly appreciated.

Thank you for your time.

frankbaele’s picture

Category: bug » support

first im moving this to support request

and second can you give some more info, because is still don't get what you are trying too acomplish.

first screen shot shows me a close menu and second shows me an open menu, I assume that the menu shouldn't be open like in the second screen shot. The best guess i have is that at certain breakpoints the boostrap css changes the display mode of you dom element. but still shooting in the dark here.

http://jsfiddle.net/ can help us more

joshuautley’s picture

The only way I was able to get the nav to collapse was to edit the bootstrap-responsive.css @media at the end of the css file from (max-width 979px) to what we needed (max-width 1154px). I would think that I should be able to override it in my sub-theme css but I couldn't get it to work with an override so I was forced to update the base theme css. This means if I need to go and update the theme I will have to remember to edit that css file in the base theme again.

markhalliwell’s picture

Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (won't fix)

Scripted response: This issue has been closed while cleaning up the issue queue. This has likely already been addressed in 7.x-3.x. If it has not, please create a new issue describing the exact issue with that version. The 7.x-2.x branch currently only receives security fixes.