-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Use new EsCard and EsLinkCard components #542
Conversation
@pichfl can you update to resolve the merge conflicts please? thank you! 👍 |
Done. It still requires a fresh release of the style guide, @mansona should be able to create one and add the change to this. |
Might have missed some of them
I would be happier to merge ember-learn/ember-styleguide#280 before the next release of the styleguide 👍 happy to update and review this once that is done |
Files that got Bigger 🚨:
Files that got Smaller 🎉:
Files that stayed the same size 🤷:
|
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 good 👍
Files that got Bigger 🚨:
Files that got Smaller 🎉:
Files that stayed the same size 🤷:
|
Might have missed some of them, also depends on ember-learn/ember-styleguide#279 and an update of the style guide /cc @mansona