Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Imports with valid, but non-stylistic syntax are not processed #350

Closed
kasper opened this issue Aug 29, 2014 · 1 comment
Closed

Imports with valid, but non-stylistic syntax are not processed #350

kasper opened this issue Aug 29, 2014 · 1 comment
Assignees
Milestone

Comments

@kasper
Copy link

kasper commented Aug 29, 2014

I’m building a tool and while testing clean-css I encountered that not all of these valid (according to W3C CSS), but clearly non-stylistic @import-syntaxes seem to be supported.

/* Imports with non-stylistic syntax */

    @import 'a.css';
@import'b.css';
@import    'c.css';
@import
'd.css';
@import '    e.css';
@import 'f.css    ';
@import 'g.css'    ;
@import 'h.css'
;
@IMPORT 'i.css';

    @import url('j.css');
@import    url('k.css');
@import
url('l.css');
@import url(    'm.css');
@import url(
'n.css');
@import url('    o.css');
@import url('p.css    ');
@import url('q.css'    );
@import url('r.css'
);
@import url('s.css')    ;
@import url('t.css')
;
@IMPORT URL('u.css');

For example, combined with the following media query imports, the processed output is mangled invalid.

/* Imports with media queries */

@import url('aa.css') screen;
@import url('bb.css') screen, handheld;
@import url('cc.css') screen and (orientation: landscape);

@import 'dd.css' screen;
@import 'ee.css' screen, handheld;
@import 'ff.css' screen and (orientation: portrait);
@kasper kasper changed the title Imports with valid, but non-stylistic syntax are not resolved Imports with valid, but non-stylistic syntax are not processed Aug 29, 2014
@jakubpawlowicz
Copy link
Collaborator

Good catch. Fix is coming!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants