-
Notifications
You must be signed in to change notification settings - Fork 14
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
Remove silent mode variable for spec v2 components #121
Conversation
f7af7b1
to
a89c7a2
Compare
@if not mixin-exists('oComponentName') { | ||
@error 'Could not compile sass as o-component-name does not have a primary mixin. ' + | ||
'See the specification [... etc]'; | ||
} | ||
|
||
@include oComponentName(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is really nice, the build service v3 should use this implementation
We would like to avoid expanding work required to complete the bower to npm migration
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks great
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
…i spec v2. Instead the primary mixin is now required: Financial-Times/origami#121
Origami spec v2 components will require a primary mixin and will no longer recommend the silent mode variable. Related proposal: Financial-Times/origami#121 Related spec suggestion: Financial-Times/origami-website#273 (comment)
Origami spec v2 components will require a primary mixin and will no longer recommend the silent mode variable. Related proposal: Financial-Times/origami#121 Related spec suggestion: Financial-Times/origami-website#273 (comment)
Origami spec v2 components will require a primary mixin and will no longer recommend the silent mode variable. Related proposal: Financial-Times/origami#121 Related spec suggestion: Financial-Times/origami-website#273 (comment)
👉 view rendered proposal