Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

Slide over nav (bug 1152573) #1153

Closed
wants to merge 2 commits into from
Closed

Conversation

mstriemer
Copy link
Contributor

Most of this was done by @chuckharmston.

r? @ngokevin

@ngokevin
Copy link
Contributor

Clicking on the top portion of the overlay (where it covers the header) doesn't close the nav.


nav--visible() {
.mkt-nav--visible & {
body.mkt-nav--visible & {
Copy link
Contributor

Choose a reason for hiding this comment

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

I will probably move these classes to the wrapper, don't think we need them on the body anymore

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@ngokevin
Copy link
Contributor

r+wc

@mstriemer mstriemer closed this in 7d95690 Apr 21, 2015
@mstriemer mstriemer deleted the slide-over-nav-1152573 branch April 21, 2015 21:22
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.

None yet

3 participants