-
Notifications
You must be signed in to change notification settings - Fork 159
Enhancement: Use a collapsable Twitter Bootstrap navbar #426
Conversation
2444a53
to
c838a0d
Compare
c838a0d
to
303f6f4
Compare
👍 |
1 similar comment
👍 |
<div class="navbar-header"> | ||
<button type="button" class="navbar-toggle collapsed" data-toggle="collapse" data-target="#bs-example-navbar-collapse-1"> | ||
<span class="sr-only">Toggle navigation</span> | ||
<span class="icon-bar"></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What are these exactly?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This stuff is basically the infamous 🍔 icon right here:
For reference, this mostly matches the first example here: http://getbootstrap.com/components/#navbar
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, so 3 repeated <span/>
: comment it for clarity, maybe? :)
Shall we use a black one and use the same link colours to make the step a bit smaller? |
Not following the question, can you rephrase it? |
Ok, instead of the shall we use an as it is closer to the current navigation? |
I think this one is fine. Waiting for travis and then I'm merging :) |
Enhancement: Use a collapsable Twitter Bootstrap navbar
👍 |
Thank you, @Ocramius! |
This PR
Signed out
Before
After
Signed In
Before
After
Smaller Screens
Before
After