search

oddbird / susy

Pre-grid responsive layout toolkit for Sass, now deprecated
http://oddbird.net/susy/
BSD 3-Clause "New" or "Revised" License
3.87k stars 347 forks source link

+susy in body and line-height problems #2

Closed agibralter closed 13 years ago

agibralter commented 14 years ago

Is there a reason that you instruct people to do:

body
  +susy

rather than just:

+susy

=susy contains a body definition and leads to body body { ... }

It also causes compass's reset line-height: 1 to win out over susy's +establish-baseline

/* line 20, ../../../vendor/bundler_gems/ruby/1.8/gems/compass-0.10.0.pre5/frameworks/compass/stylesheets/compass/utilities/general/_reset.sass */
body {
  line-height: 1;
  color: #000;
  background: #fff;
}

...

/* line 27, ../../../vendor/bundler_gems/ruby/1.8/gems/compass-susy-plugin-0.7.0.pre4/sass/susy/_vertical_rhythm.sass */
body body {
  font-size: 100%;
  line-height: 1.5em;
}
mirisuzanne commented 14 years ago

Sorry, That's a discrepancy between the live tutorial written for the last stable release (0.6.3) and the version of Susy you are using (0.7.0pre4). The new tutorial will be up to date as soon as we have a stable 0.7.0, but I must have missed that in the changelog as well. I'll make sure to fix that. Thank you for the catch.

Cheers, Eric

mirisuzanne commented 13 years ago

fixed