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.

Sign up for GitHub

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

Jump to bottom

Conflicts with http://include-media.com/ #27

Open
pryley opened this issue Jul 18, 2016 · 1 comment
Open

Conflicts with http://include-media.com/ #27

pryley opened this issue Jul 18, 2016 · 1 comment
Labels
backlog enhancement

Comments

@pryley
Copy link

pryley commented Jul 18, 2016

I suggest you change/rename the media mixin to mt-media or something similar as it conflicts with the popular include-media media mixin.

https://www.npmjs.com/package/include-media
@tbredin
Copy link
Contributor

tbredin commented Jul 19, 2016

We're looking at reworking the media query setup very soon which will involve reviewing naming, see #26. Adding a namespace to our mixins is an option we would consider

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backlog enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@pryley @tbredin @thebenshaw