Removing "@import url();" from css file

[moje-s]

After css file minification there is missing the "@import url();".

[CaffeineOnIce]

can you explain a bit more? Is it for every minification? is the import not working?

[moje-s]

Whole part of code "@import url('dir/file.css');" is missing, like it was removed while minification.

Yes, it happens every time.

[CaffeineOnIce]

@owhs can you check if it occurs with you? I can't seem to replicate in mine

[owhs]

Yes, I could replicate.

By logging the output, it says:

"errors":["Ignoring local @import of \"app.css\" as resource is missing."]

Fixed by adding in the CSS settings.json:

"inline": ["none"]

My css settings now read:

"autominify.cssMinifierOptions": {
    "inline": ["none"],
    "level": {
        "1": {
            "all": true
        }
    }
}

And works as expected 😊
Let me know if this resolves it for you @moje-s

[owhs]

Whole part of code "@import url('dir/file.css');" is missing, like it was removed while minification.

Yes, it happens every time.

By doing the above steps, it will just include the @import lines "as is", there's a method to "inline" the imports, but personally I wouldn't like that behaviour, hopefully you agree :)

[CaffeineOnIce]

@moje-s check if owhs' method works for you. if it does, do close the issue.

[owhs]

@moje-s check if owhs' methods works for you. if it does, do close the issue.

SSSfine for now, but for the record, the options break when you chose when you chose level 0 or 2,
At a guess, you're doing an object.assign from the default optionObj with the user options
So it still keeps the default values for level 1

As I said, it's not really an "issue" - more nuanced if people wanted to use the other levels, as 1 is fine for 99% use cases :)
No point spending time looking into fixing an issue that nobody experiences, but if you ever have someone mention it, that's the behaviour behind it.

[moje-s]

"autominify.cssMinifierOptions": {
    "inline": ["none"],
    "level": {
        "1": {
            "all": true
        }
    }
}

Almost good, result is without ' or " in path - @import url(dir/file.css);
I tested it for both ' and ".

[CaffeineOnIce]

wait, level 1 doesn't get removed? It should work. Did you remove the entire level 1 code and replaced it with level 2?

[moje-s]

Still the same.

[CaffeineOnIce]

@owhs mmm. Weird. Replacing the level 1 code for level 2 works for me. I'll check the code tomorrow. If you find something, you can pull it. I'll merge the update.

[owhs]

Still the same.

 @import url('app.css');
 body{
     background: #fff;}

works fine, but then:

body{
    background: #fff;}
@import url('app.css');

the import disappears,
are all your imports at the very top?

if not, probs will have to push an update that logs debug info to the output panel to be able to diagnose further

[CaffeineOnIce]

yeah, I'll look into that by today. if not, by tomorrow.

[moje-s]

@charset "UTF-8";

@import url("file.css");

1st and 3rd line.

[owhs]

@charset "UTF-8";

@import url("file.css");

1st and 3rd line.

Ah it's likely the @ charset line, there'll prbs be another arg for other @ queries, I'll look tmr or over weekend some point

2 points;

1. Does it still fail if you remove the utf charset line?
2. Why do you even need to specify UTF-8? Does it break your site if you don't have that line? Do you have special characters in your CSS- if so, you should probs escape those values :)