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 350 forks source link

Variable conflict #398

Closed 0nn0 closed 9 years ago

0nn0 commented 9 years ago

New to Susy. Couldn't get the debug grid to function properly. Finally figured out that I had conflicting variables, to be specific $base-line-height: 1.5; which caused the debug-grid to malfunction. Is there a reason for not prefixing all susy-lib related variables with e.g. 'susy-' to avoid conflicts?

mirisuzanne commented 9 years ago

That's not a Susy variable — it's part of our optional Compass integration. $base-line-height is a Compass variable that we will respect if it's set.

0nn0 commented 9 years ago

Thanks for the quick clarification Eric. Good to keep in mind when I dive deeper into Susy.

krzkrzkrz commented 9 years ago

$base-line-height is also defined by default in bourbon bitters. Maybe prefixing variables is the way to go to avoid conflicts?