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

Added bottom padding to all bootstrap themes #255

Merged
merged 1 commit into from Dec 1, 2014

Conversation

@DannyBen
Copy link
Contributor

@DannyBen DannyBen commented Nov 29, 2014

As discussed in issue #251 I added padding to the main document area to all bootstrap themes.

I have updated the Base.css to have 100px padding, like this:

div.col-md-9 {
    padding-bottom: 100px;
}

Normally, I would choose to do it differently - like to assign a specific class to the sidebar and main area, but I followed the convention in the CSS, since I saw there was already this:

div.col-md-3 {
    padding-left: 0;
}

Screenshots of the Yeti theme, before (top) and after:
beforeafter

d0ugal added a commit that referenced this pull request Dec 1, 2014
Added bottom padding to all bootstrap themes
@d0ugal d0ugal merged commit 589d699 into mkdocs:master Dec 1, 2014
1 check passed
1 check passed
@tomchristie
continuous-integration/travis-ci The Travis CI build passed
Details
@d0ugal
Copy link
Member

@d0ugal d0ugal commented Dec 1, 2014

Thanks!

@d0ugal d0ugal added this to the 0.12.0 milestone Dec 4, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants