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

Update dependencies #60

Merged
merged 4 commits into from
Nov 28, 2019
Merged

Update dependencies #60

merged 4 commits into from
Nov 28, 2019

Conversation

oliverturner
Copy link
Contributor

@oliverturner oliverturner commented Nov 25, 2019

Deprecated o-overlay mixins have been added verbatim: the latest
version of oOverlay isn't compatible with implementations that attempted
to recreate behaviour component-by-component.

WIP while n-ui-foundations is being upgraded

🐿 v2.12.5

Deprecated o-overlay mixins have been added verbatim: the latest
version of oOverlay isn't compatible with implementations that attempted
to recreate behaviour component-by-component. 🐿 v2.12.5
@oliverturner oliverturner changed the title Update dependencies WIP: Update dependencies Nov 25, 2019
@import "o-overlay/main";
@import "o-visual-effects/main";

$system-code: "n-syndication";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We shouldn't be setting the system code in components, only in systems. If this is needed for demos/testing it should be put into a stylesheet used for that purpose.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added to ./test/main.scss

@oliverturner oliverturner changed the title WIP: Update dependencies Update dependencies Nov 28, 2019
@oliverturner oliverturner merged commit def8028 into master Nov 28, 2019
@oliverturner oliverturner deleted the origami-major-cascade-2019 branch November 28, 2019 16:09
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

Successfully merging this pull request may close these issues.

4 participants