Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Justified nav in navbar #14598

Closed
CydGoblin opened this issue Sep 11, 2014 · 12 comments
Closed

Justified nav in navbar #14598

CydGoblin opened this issue Sep 11, 2014 · 12 comments

Comments

@CydGoblin
Copy link

I'm using this custom build and every li looked like this
image

So I found This
image

@hnrch02 hnrch02 added the css label Sep 11, 2014
@hnrch02
Copy link
Collaborator

hnrch02 commented Sep 11, 2014

Please make your example into a JS Bin. Also, which version of Bootstrap are you using?

@CydGoblin
Copy link
Author

JS bin demo
Added .nav-justified to ul and all li inside is like stacked. I'm using latest version of bootstrap.

@hnrch02
Copy link
Collaborator

hnrch02 commented Sep 11, 2014

Pretty sure justified navs in navbars aren't supported. /cc @mdo @cvrebert

@cvrebert
Copy link
Collaborator

Correct. It's an option of Tabs and Pills, and you can add custom styles to make it navbar-like (see http://getbootstrap.com/examples/justified-nav/ ), but it can't be combined with the navbar.
I know this has come up before. It might be worth documenting this.

@hnrch02 hnrch02 changed the title nav-justified li width: 1% Justified nav in navbar Sep 11, 2014
@hnrch02 hnrch02 added the docs label Sep 11, 2014
@hnrch02 hnrch02 added this to the v3.2.1 milestone Sep 11, 2014
@mdo
Copy link
Member

mdo commented Sep 17, 2014

Nope, they're not supported.

@carasmo
Copy link

carasmo commented Sep 17, 2014

Yes, please document. Saw a ? on SO today about it.

@carasmo
Copy link

carasmo commented Sep 17, 2014

Here is a way to do it:

http://jsbin.com/sohayi/1/

It's pretty simple.

@mdo
Copy link
Member

mdo commented Sep 19, 2014

@carasmo I know we've said no new features, but would you be willing to open up a PR with perhaps a new example for that?

@carasmo
Copy link

carasmo commented Sep 19, 2014

@mdo ... Right after I posted it I thought that something like this should not be added because it requires use of the responsive utilities to hide the brand and show it again and, well, justified navs are just not that popular. Never needed them since 99 in my own life. But that's just me. Bootstrap is already full enough. However, if you want, I will try -- never have made a pr in my life.

@mdo
Copy link
Member

mdo commented Sep 23, 2014

Punting to v4.

@mdo mdo closed this as completed Sep 23, 2014
@mdo mdo removed this from the v3.2.1 milestone Sep 23, 2014
@cvrebert
Copy link
Collaborator

Due to the confusion caused by the nav in the name, it's worth documenting/clarifying that justified navbars don't exist (out of the box, anyway), IMO. It's come up several times.

@cvrebert cvrebert reopened this Sep 23, 2014
@mdo
Copy link
Member

mdo commented Sep 23, 2014

Relevant previous requests:

Plus, all the issues around the Justified Nav example we have, which is a janky implementation of a responsive navbar.

@mdo mdo closed this as completed in 5d05173 Sep 23, 2014
cvrebert added a commit that referenced this issue Sep 27, 2014
Saranya-Raaj pushed a commit to Saranya-Raaj/bootstrap that referenced this issue Oct 9, 2014
Saranya-Raaj pushed a commit to Saranya-Raaj/bootstrap that referenced this issue Oct 9, 2014
@mdo mdo mentioned this issue Aug 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants