@media

[JonDum]

So @media tags aren't rendering so well right now.

input:

@media only screen and (max-width: 998px) {
  .page.grid {
    padding: 0% 4%;
  }
}

output:

@media
  only
    screen
      and
        (max-width
          &:
            998px)
              .page.grid {
    padding 0% 4%

It looks like the code is treating the spaces as additional selectors because it's getting through to line 109 selectors = selector.split(/[\s]+/);

Maybe we should add another check to the if statement on line 99 to see if the selector starts with @media then process the contents of that block recursively?

The way it is now doing that recursively would require a lot of rewriting. Maybe there's a better way. What are your thoughts?

[dciccale]

I think your solution is a good one, we could make a method to sub-process that line of the @media and return the stuff. So that we have to call that method inside the ìf` you are talking and things should keep simple.

Anyway I will think about other solutions.. thanks

[JaKXz]

I'm seeing this issue as well.

[dciccale]

Seems a pretty good fix to do by adding an additional check. Didn't have the time yet to implement it. I will try it whenever I have time for this, please feel free to contribute

[KyleKing]

I'm still having this issue as well, given:

@media(min-width:1200px) {
    .timeline>li {
        min-height: 170px;
    }
}

Css2stylus outputs:

@media(min-width
  &:1200px)
    .timeline>li {
        min-height 170px

[ghost]

Still having this issue too

@media
    (max-width
      &:
        767px)
          before {
        left 40px

ul
  &.timeline
    & > li
      margin-bottom 20px
      position relative
      width 100%
      float left
      clear left

[dciccale]

this needs to be addressed in a special case. open to contributions :)

will try to fix it at some point

[JonDum]

Honestly, it's not a feature, but a straight up bug. :|

[dciccale]

well as I see it is a feature. because this version i never made any special implementation for handling media queries. so if support is something is needed to be implemented (not fixed) is a feature. not a bug

[dciccale]

and what i meant above in my previous comment by "fix" was about fixing the github issue by implementing this feature.

thanks for the contribution

[NeXTs]

+1
I also faced @media issue

[martpie]

Hi, any news on this ?

[dciccale]

i am pretty busy to handle this now, please feel free to contribute.

On Wed, Oct 28, 2015 at 10:08 AM, Pierre de la Martinière <
notifications@github.com> wrote:

Hi, any news on this ?

—
Reply to this email directly or view it on GitHub
#7 (comment)
.

[KyleKing]

I just submitted a pull request that should fix this issue