CSS problem in demo build

[bill]

Doing a build like:

./build.sh copyTests=true mini=false action=clean,release profile=demos-all cssOptimize=comments.keepLines cssImportIgnore=../dijit.css

And then accessing the form demo:

demos/form/demo.html

It doesn't display correctly. Apparently dijit.css (which is loaded through tundra.css) didn't get loaded. Not sure why the demo fails and other demos work.

[Douglas Hays]

I noticed that @import for dijit.css was positioned after some initial rules. The @import has to be before any rules (per css2 spec) so maybe that's causing it to be ignored.

[Rawld Gill]

doug is correct. The port of the v1.6- CSS optimizer (in util/build/transforms/optimizeCss.js) is improperly excluding CSS interning according to the cssImportIgnore directive while recursing through nested CSSs. In fact, cssImportIgnore can be applied only to @import's in a top-level CSS.

Also noted this error was showing up on other demos (e.g., mail).

Also noted jpg and gif files were not being copied as binary if transformed by read-write instead of copy. This can happens when a resource is tagged by something like "test" (e.g., dijit/tests/images).

[Rawld Gill]

(In [25559]) fixed improper exclusion of nested CSS according to cssImportIgnore; fixed jpeg and gif files not being copied as binary; fixes #13215; !strict