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

remove 60px padding from body top #37

Closed
ShaggyDude opened this Issue Mar 29, 2012 · 3 comments

Comments

Projects
None yet
2 participants
@ShaggyDude

ShaggyDude commented Mar 29, 2012

it jacks up the menu at 768px;

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Apr 9, 2012

no dont remove it!

i think i know this i had a simular problem with bootstrap ( my own stuff not this particular wordpress theme). you have to load responsive css at last after all others. like in this example http://twitter.github.com/bootstrap/examples/hero.html

<link href="../assets/css/bootstrap.css" rel="stylesheet">
<style type="text/css">
  body {
    padding-top: 60px;
    padding-bottom: 40px;
  }
</style>
<link href="../assets/css/bootstrap-responsive.css" rel="stylesheet">

if you load the responsive stuff before that, the padding will not be removed for < 786px !
if you just get the customizes css for twitter bootstrap you get only one file (sadly) and when i tryed to make it like the example i got this issue. so i guess this is what happened here just without looking at the code.

ghost commented Apr 9, 2012

no dont remove it!

i think i know this i had a simular problem with bootstrap ( my own stuff not this particular wordpress theme). you have to load responsive css at last after all others. like in this example http://twitter.github.com/bootstrap/examples/hero.html

<link href="../assets/css/bootstrap.css" rel="stylesheet">
<style type="text/css">
  body {
    padding-top: 60px;
    padding-bottom: 40px;
  }
</style>
<link href="../assets/css/bootstrap-responsive.css" rel="stylesheet">

if you load the responsive stuff before that, the padding will not be removed for < 786px !
if you just get the customizes css for twitter bootstrap you get only one file (sadly) and when i tryed to make it like the example i got this issue. so i guess this is what happened here just without looking at the code.

@ShaggyDude

This comment has been minimized.

Show comment
Hide comment
@ShaggyDude

ShaggyDude Apr 11, 2012

OK,
It has also been suggested to move the admin menu to the bottom with something like this.
http://wordpress.org/extend/plugins/stick-admin-bar-to-bottom/

ShaggyDude commented Apr 11, 2012

OK,
It has also been suggested to move the admin menu to the bottom with something like this.
http://wordpress.org/extend/plugins/stick-admin-bar-to-bottom/

@rachelbaker

This comment has been minimized.

Show comment
Hide comment
@rachelbaker

rachelbaker Apr 11, 2012

Owner

Thanks for catching this - I just noticed it myself when using the theme.

This has been fixed in the v.86 release with an addition to the style.css file.

Owner

rachelbaker commented Apr 11, 2012

Thanks for catching this - I just noticed it myself when using the theme.

This has been fixed in the v.86 release with an addition to the style.css file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment