Using em instead rem about sassy-gridlover

[hiulit]

Thanks for the appreciation! I'm glad you find it useful.
About adding em support, yeah, why not. I didn't think of it because I personally don't use them that much, but the Gridlover app uses them and since this is kind of a "Sass port" of their app, it would be consistent to add support to ems.
What I think would be the best approach, would be to let the users decide which unit they'd like to output. For example:

• px
• rem
• px/rem (rems with pixels fallback)
• em

This could be achieved via variables, as it is working right now with the rem support. What do you think?
I could try and give it a try this week. But if you would like to help, don't hesitate!
Thanks!

[alekspetrov]

Oh, It maybe a wrong question, I asked how to re-config it for ems ) cause I need flexible config for project and I don't want to change my setup with the GridLover.app. I'll see it later, have no free time now.

[hiulit]

Hi,
I've been trying to add em support but I haven't been successful due to the relativity of ems. I can't figure out the way to do it...
I've created a branch for testing em support https://github.com/hiulit/Sassy-Gridlover/tree/em-support

[alekspetrov]

Yea, first time I tried to replace rem by em and that was the reason to make an issue =) cause output was another size.

I think all calculations have to be in px val, then translate the output into em or any other vals. It gives more flexible config for the future, for example if somebody want to support vh or vw.

[hiulit]

I've been in contact with the Gridlover app team and we'll try to figure it out.

[alekspetrov]

Great, let me know.

[cibulka]

+1!

[hiulit]

Quick update. I've been in contact with one of the Gridlover developers but unfortunately neither of us have the time right now to take a deep look into this matter. So any help is extra welcome :)

[hiulit]

There's a branch for this issue https://github.com/hiulit/Sassy-Gridlover/tree/em-support

[hiulit]

Good news everyone!

@walmokrani help a lot with this issue #7. I have to take a look at it so I can merge it. But it's looking good :D

[hiulit]

Hey guys @alekspetrov @cibulka !
I've just released v2.0.0 with em support! Check it out and tell me what you think or if it works for you! ;)

Thanks again to @walmokrani !