Added pxToEm in _mixins.scss

Wakkos / Wakkos-CSS-Framework

SCSS Framework para agilizar maquetación Frontend.

http://twitter.com/wakkos

148 stars 160 forks source link

Closed AntonioNav closed 9 years ago

AntonioNav commented 10 years ago

This function is already in use in _tipografia.scss Seems to be missing in this file.

Wakkos commented 10 years ago

Esta función la quería quitar, porque no veo su utilidad teniendo el mixin de font-size().

Que opinas?

AntonioNav commented 10 years ago

En _tipografia.scss se está usando para calcular el margen inferior:

    margin : 0 0 $baselineheight - pxToEm($gutterwidth);

Esto con el mixing no se me ocurre cómo se podría hacer, habría que buscar otro método de calcular ese margen.

Quizá lo más correcto sería que la variable $gutterwidth se definiese en 'em' en vez de usar 'px'.

Wakkos commented 10 years ago

No, porque para la relación de márgenes en vertical, uso $baselineheight, entonces sustituimos eso por $baselineheight?

AntonioNav commented 10 years ago

Me parece bien, así quedaría más simple.

Wakkos commented 10 years ago

Pues.. adelante!