Closed arohner closed 9 years ago
Wow, that is a peculiar use of url
- targeting a DOM element. Even IE has stopped supporting it. I'm inclined to say "won't fix", in particular because behavior
URLs can also be actual URLs - and because load-css doesn't do any CSS-parsing (it leaves that to clean-css). I guess the code could know that, while "#foo" is a valid relative url, it does not make sense as a CSS url, so leave it alone.
Any thoughts?
I came to the conclusion that it was safe to just leave URLs starting with #
alone. 0.17.2
is released. Please let me know if that solved your issue.
I came to the conclusion that it was safe to just leave URLs starting with # alone
That's the solution I was going to propose. Looks like 0.17.2 works for me. Thanks for the quick turnaround!
:+1:
Try to bundle https://github.com/mapbox/mapbox.js/ 's css, which contains:
.foo { behavior:url(#default#VML); }
[1]When trying to load the bundle, you'll get an exception like:
It looks like what's happening is that optimus.assets.load-css first converts
url(#default#VML)
to an absolute url:"/bower/mapbox.js/#default#VML"
, and then callsremove-url-appendages
which leaves"/bower/mapbox.js/"
as an asset path. Code downstream tries to slurp, and throws the 'is a directory' exception.[1] - This appears to be an IE CSS extension: http://stackoverflow.com/questions/26339276/what-is-behavior-url-property-in-css