search

postcss / postcss-nested

PostCSS plugin to unwrap nested rules like how Sass does it.
MIT License
1.16k stars 66 forks source link

fix order of rules #52

Closed Riim closed 7 years ago

Riim commented 7 years ago

Исправление для сломавшегося порядка следования правил.

ai commented 7 years ago

Can you add a test for this fix (I am not sure what order do you try to fix).

ai commented 7 years ago

Released in 2.0.1

ai commented 7 years ago

Thanks :)

skipjack commented 7 years ago

@ai still seeing this issue, can you see my configuration in postcss/postcss-loader#210 to see if anything is off? I can't tell if the issue is this plugin or the loader...

ai commented 7 years ago

Can you describe you exactly problem here? Input CSS, output, expected output?

(Today is hard day for me, so I need you help)

skipjack commented 7 years ago

Yeah, no worries. I'm actually going to put together a minimal test case for this in the issue I linked to above so I'll cc you once I have that ready.

skipjack commented 7 years ago

@ai see the demo I just put up.

ai commented 7 years ago

Thanks, I will look tomorrow.

ai commented 7 years ago

@skipjack I tested with latest postcss-nested and output was:

body {
    margin: 0;
}

.test {
    display: flex;
    font-size: 3em;
    height: 100vh;
    justify-content: center;
    align-items: center;
    color: black;
    background: grey;
}

.test--white {
        color: white;
    }

.test--blue {
        color: blue;
    }

Very strange that we have different behaviour in your demo.

skipjack commented 7 years ago

Ah thanks for testing... it actually turned out to be postcss-cssnext, sorry for bugging you!