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

Merging mixed rules | Order bug #21

Open
AaronLayton opened this issue Sep 17, 2014 · 1 comment
Open

Merging mixed rules | Order bug #21

AaronLayton opened this issue Sep 17, 2014 · 1 comment

Comments

@AaronLayton
Copy link

It would be really good if CSSShrink could figure out the `@media' rules in this snippet and merge them together. It seems to have an issue when they are in this order but if you put both the 768's together then it picks up on it (I am testing on http://cssshrink.com/ )

header {
  color: red;
}
@media only screen and (min-width: 768px) {
  header {
    color: green;
  }
}
@media only screen and (min-width: 1024px) {
  header {
    color: blue;
  }
}

section {
  color: green;
}
@media only screen and (min-width: 768px) {
  section {
    color: blue;
  }
}
@media only screen and (min-width: 1024px) {
  section {
    color: red;
  }
}
@AaronLayton AaronLayton changed the title Merging rules Merging mixed rules | Order bug Sep 17, 2014
@yairEO
Copy link

yairEO commented Apr 27, 2015

I recommend this - https://www.npmjs.com/package/gulp-combine-media-queries

Also the title is wrong. it should be Merging mixed media queries | Order bug
because this ticket is only about media queries.

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

2 participants