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

Website: Fix buggy behavior in nav and add global message #1190

Merged
merged 2 commits into from
Sep 30, 2016
Merged

Conversation

magwalk
Copy link
Contributor

@magwalk magwalk commented Sep 30, 2016

Fixes the bug where the nav overlaps with the footer on short screen sizes and adds a global message bar that can be modified/updated as needed.

screen shot 2016-09-30 at 9 40 26 am

screen shot 2016-09-30 at 9 40 37 am

Adds a global message that will appear above the nav that can
be customized to advertise upcoming events.
@cwebberOps
Copy link
Contributor

@vjeffrey or @chris-rock can you take a quick look over this and approve if you are good with the changes?

@magwalk Let's give the inspec team a chance to take a glance at this before we merge it.

@vjeffrey
Copy link

yup! i'll take a look in a few minutes.

@vjeffrey
Copy link

vjeffrey commented Sep 30, 2016

👍 looks great! thanks @magwalk @cwebberOps

while I have your attention, could one of you do me a favor and take a look at/test #1191. the tutorial part of the site doesn't currently work with safari, that pr should fix it.

^^ nevermind about testing my pr. looks like there's still errors. grrrrr safari.

@magwalk
Copy link
Contributor Author

magwalk commented Sep 30, 2016

Thanks @vjeffrey!

@magwalk magwalk merged commit bf623e6 into master Sep 30, 2016
@magwalk magwalk deleted the mw/nav branch September 30, 2016 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants