Depreciate original mixin about modularscale-sass HOT 11 CLOSED

I think you're right. I was thinking it was there for people who are more comfortable with mixins, but that's kind of a crutch, right?

from modularscale-sass.

Functions are better, they should learn them. Lets get rid of it.

from modularscale-sass.

If we do, don't forget that the classes mixin uses this https://github.com/scottkellum/modular-scale/blob/master/stylesheets/_modular-scale.sass#L208

from modularscale-sass.

I was about to delete it but then saw the classes mixin and the regular mixin next to each other. Might as well leave it in as they have a lot of similarities.

from modularscale-sass.

It's still a bit vestigial. I'm still thinking of taking it out, and rewriting the classes mixin accordingly. Should be a pretty simple change.

from modularscale-sass.

Yea, the classes mixin can get re-written accordingly to use the function instead.

from modularscale-sass.

I went ahead and deprecated it. For now running the mixin will send a warning to the console, but still work.

from modularscale-sass.

Which brings up a topic we should mention for extension devs, when to use @debug and @warn.

from modularscale-sass.

So many new things to learn.

from modularscale-sass.

good point. I'd love to draft something on that. I'd be lost w/o @debug

from modularscale-sass.

I have a start to a guide on if, for, each, while that has some uses of this in the code samples but nothing that dives into the subject. Take a peek at the read lists and feel free to add to them :)

from modularscale-sass.