Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix(docs): Fix nav list bar css #3363

Closed
wants to merge 1 commit into from

Conversation

ksheedlo
Copy link
Contributor

Fixes #3262.

/cc @matsko could you take a look at this and make sure it doesn't break anything?

@matsko
Copy link
Contributor

matsko commented Jul 26, 2013

Will do. Thank you for putting it together.

@matsko
Copy link
Contributor

matsko commented Jul 26, 2013

Just took a look, turns out that you do need hidden since it makes the elements stay inside an element that has a border radius:

Here's the correct CSS code, just replace the code inside of animations.css with this:

.nav-list {
  padding:0;
}

.nav-list li {
  margin:0!important;
  padding:0 15px;
  height:20px;
  overflow:hidden;
}

@ksheedlo
Copy link
Contributor Author

I'm not sure why that works, but it does.

@petebacondarwin
Copy link
Contributor

LGTM. By the way, you don't capitalize the first character of the commit message subject.

@ksheedlo ksheedlo closed this in 3570005 Jul 27, 2013
@ksheedlo
Copy link
Contributor Author

Noted. Thanks, Pete.

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.

Nav list bar styles broken in docs
3 participants