Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

v2 #20

Merged
merged 6 commits into from
Oct 24, 2015
Merged

v2 #20

merged 6 commits into from
Oct 24, 2015

Conversation

iamphill
Copy link
Owner

@iamphill iamphill commented Oct 2, 2015

Moved the offcanvas classes off the navbar element to the element that holds the navbar links. This leaves the navbar-header on the screen at all times.

Also looking at improving the transitions between hidden & shown.

Going to also look at improving the styles all together. With a breaking release, might as well go all out!

Will close #19

Moved the offcanvas classes off the navbar element to the element that holds the navbar links. This leaves the navbar-header on the screen at all times.
Also looking at improving the transitions between hidden & shown
@target.on 'transitionend', =>
console.log @target.is ':not(.in)'
if @target.is ':not(.in)'
@target.height ''

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line contains inconsistent indentation

A lot of dropdown CSS now falls back to Bootstraps - rather than overriding
@iamphill iamphill added this to the v2 milestone Oct 2, 2015
iamphill added a commit that referenced this pull request Oct 24, 2015
@iamphill iamphill merged commit 0c71b47 into master Oct 24, 2015
@iamphill iamphill deleted the v2 branch June 8, 2018 18:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Keep bar with brand in canvas, move only navigation-DIV and search off-canvas
2 participants