Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

shorter mixin call #2178

Closed
bmargogne opened this issue Oct 28, 2016 · 4 comments
Closed

shorter mixin call #2178

bmargogne opened this issue Oct 28, 2016 · 4 comments

Comments

@bmargogne
Copy link

Hi. This is a suggestion more than an issue but I am not sure of where to suggest it ...

Would it be possible to call mixins using a syntax such as

@mixinName( ... )rather than @include mixinName( ... )?

That would be a lof of typing saved !

Thank you for the great work.

@joneff
Copy link

joneff commented Nov 7, 2016

Just for the sake of argument, lets imagine you have a mixin called media. How would the compiler know when you want to use media queries and when to use your mixin?

I'd welcome shorter mixins calls, as long as they are still explicit enough, do not interfere with css syntax and are not magical.

@bmargogne
Copy link
Author

Hello, and thank you for your reply.

I am not an experienced developper, so I will surely not have an
argument with you :)

I didn't think this would be a serious issue : I thought such new
writing convention would imply a list of forbidden mixin names (extends,
include, media...), just like any other language do (I guess names like
'function' or 'class' are forbidden in many languages).

I do not really know all implications apparently !

have a nice day.

Le 07/11/2016 à 13:17, Иван Жеков a écrit :

Just for the sake of argument, lets imagine you have a mixin called
|media|. How would the compiler know when you want to use media
queries and when to use your mixin?

I'd welcome shorter mixins calls, as long as they are still explicit
enough, do not interfere with css syntax and are not magical.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#2178 (comment), or
mute the thread
https://github.com/notifications/unsubscribe-auth/ALuPtSP9H1eaBsja9nJoR2ZM-AR3uZGWks5q7xbugaJpZM4KjWJn.

Bertrand MARGOGNE
+33 6 99 55 75 43

@long-lazuli
Copy link

long-lazuli commented Nov 9, 2016

What about @(mixinName) { ... } or simply take the Sass syntax +mixinName { ... }.
But I this has been already discussed...
#366

@joneff
Copy link

joneff commented Nov 17, 2016

You are right. This thread should be closed.

/cc @chriseppstein

@nex3 nex3 closed this as completed Dec 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants