At rule fixes

[emmatown]

What:
Change the way at rules are joined

Why:
We had two plugins for insertion and it fixes bugs with non-nested at rules

How:
https://github.com/thysultan/stylis.js/blob/master/stylis.js#L339-L356

Checklist:

• Documentation N/A
• Tests
• Code complete

cc @giuseppeg

[codecov]

Codecov Report

Merging #326 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #326      +/-   ##
==========================================
+ Coverage   96.49%   96.53%   +0.03%     
==========================================
  Files          16       16              
  Lines         571      577       +6     
  Branches      138      138              
==========================================
+ Hits          551      557       +6     
  Misses         15       15              
  Partials        5        5

Impacted Files	Coverage Δ
packages/emotion/src/index.js	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 272f5b2...a96963f. Read the comment docs.

[giuseppeg]

@mitchellhamilton cool, I had to fix a few more bugs.. eg @import @charset are ignored in your patch. https://github.com/zeit/styled-jsx/blob/d4e21c38797da2cd5529953a617190abbcc0447b/src/lib/style-transform.js#L92

There are still a few edge cases where it fails but I might ignore them for now.

[emmatown]

@giuseppeg Just had a quick look at your fixes, I'm pretty sure they would break if there was more than just the @import or @charset in the css because the -2 context is when the css is done processing and the content has all of the css. I think it would work if it was like this

if (context === 1) {
  if (content.charAt(0) === '@') {
    splitRulesQueue.push(content)
    return ''
  }
}

[giuseppeg]

oh right silly me, thanks for the heads up