componentjs / builder2.js

builder for component
50 stars 20 forks source link

Rewrite url remove unnecessary url() wrap #49

Closed azhang closed 10 years ago

azhang commented 10 years ago

rewriteCSSURLs wraps url() already.

azhang commented 10 years ago

Previously it'd do

  src: url('url(//netdna.bootstrapcdn.com/font-awesome/4.0.3/fonts/fontawesome-webfont.eot?v=4.0.3)');

now the same line becomes

  src: url('//netdna.bootstrapcdn.com/font-awesome/4.0.3/fonts/fontawesome-webfont.eot?v=4.0.3');
jonathanong commented 10 years ago

oh shoot. i wonder how all my tests pass and my stuff works. can you add a test?

jonathanong commented 10 years ago

should've pinged me that you added a test!

azhang commented 10 years ago

Ah, will ping next time :)