Consider change padding-top in Sample for Bootstrap fixed-top navbar #33
Unanswered
surrealist
asked this question in
Q&A
Replies: 2 comments
-
|
Thanks! Yes we want this to be better than the Microsoft template by far! I'm in the middle of a major update and I will merge your pull request as soon as I can. |
Beta Was this translation helpful? Give feedback.
0 replies
-
|
Ok, I've now given this a go and I can see why they chose 50px. It's so that the carousel touches the bottom of the nav bar. The docs you linked says "Try out your own values", hence the 50px. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Please consider change
<body>padding-top from 50px to 70px in the Sample projectsBootstrap 3 document states this in section Fixed to Top
http://getbootstrap.com/components/#navbar-fixed-top
I found padding-top: 50px in the following places:
Content/Site.cssandContent/Site.less)Styles/site.scss)I think this is not actually related to core of the boilerplate. But when apply correct padding-top as suggested by Bootstrap, it will has space between navbar and jumbotron (or other page contents) and that make it really looks better.
// As everyone knew, default ASP.NET MVC template from Microsoft uses 50px too, but this boilerplate aims to be better than the default template, doesn't it? 😏
Please tell me if I should send send you a pull request.
Beta Was this translation helpful? Give feedback.
All reactions