Skip to content

Conversation

@mdo
Copy link
Member

@mdo mdo commented Aug 13, 2013

screen shot 2013-08-12 at 9 39 27 pm

This overhauls our navbars to address a few items:

  • Navbars are now responsive by default (fits better with mobile first approach)
  • Navbars now require .nav-header to group .navbar-brand and .navbar-toggle for proper mobile display in conjunction with the collapsed content.
  • Renamed .nav-collapse to .navbar-collapse for a more consistent naming convention.
  • Refactored dropdown menus to use the old 2.x approach (no longer absolutely positioned, but rather as navbar nav links). Fixes Responsive navbar - inconsistent alignment in dropdown menus #8792.
  • Simplified examples in docs to only highlight content not displayed in the first navbar example.

It can add some heft to an otherwise lightweight navbar, but I think all these improvements help the component and make it more durable in the long run. Still needs some finessing, but it's close I think.

mdo added 3 commits August 12, 2013 21:23
* Now responsive by default (fits better with mobile-first approach)
* Requires `.nav-header` to group `.navbar-brand` and `.navbar-toggle`
for proper mobile display.
* Changed `.nav-collapse` to `.navbar-collapse`
* Simplified examples in docs
Conflicts:
	components.html
	dist/css/bootstrap.min.css
@mdo
Copy link
Member Author

mdo commented Aug 13, 2013

Also addresses #9303.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants