inlined imports should be rebased to current file

[jessebluemr]

My problem is related to #42

Problem description

File import.css:

@import url(../otherdir/imported.css);
p.import
{
    background: url(import.png);
}

File otherdir/imported.css

p.imported
{
    background: url(import2.png);
}

Will fail by:

gulp.src(['dir/import.css'])
.pipe(cleanCSS({}))
.pipe(gulp.dest("out"))

with an Exception that ../otherdir/imported.css can not be resolved.

I had expected an outcome like this:

minified 'import.css':

p.imported{background:url(../otherdir/import2.png);}
p.import{background:url(import.png);}

I was not able to fix this problem by configuration, because the path of the current file is required to correct the rebaseTo option.

Solution

This fix always transports the file.path together with the contents to CleanCss.minify, which enables the imports of relative files.

A second problem occurred when inline: ['local'] is required. The image URLs defined in the imported css files were unexpected rebased to the current working directory. I had expected a rebase relative to the current file, so that the browser can import e.g. all images relative from the minified css file. Like in the sample above.

The patch ensures that the rebaseTo option is defined as the parent directory of the current processed file if not otherwise specified.

[scniro]

@jessebluemr thank you very much for your work! I've been trying to solve this one for a bit now and I'm very pleased to see you took the time to contribute for this one. I'll publish up a new version on npm shortly. Rock on!

[scniro]

@jessebluemr 3.4.1 now out in the wild

[jessebluemr]

@scniro Thank you very much! It was a pleasure to contribute.