fix: allow spaces surrounding url

jsdom · May 3, 2021 · 0e42fb2 · 0e42fb2
1 parent 7284d37
commit 0e42fb2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/lib/CSSStyleDeclaration.test.js b/lib/CSSStyleDeclaration.test.js
@@ -437,6 +437,12 @@ describe('CSSStyleDeclaration', () => {
     expect(style.backgroundImage).toEqual('url(http://some/url/here3.png)');
   });
 
+  test('url parsing works surrounded by spaces', () => {
+    var style = new CSSStyleDeclaration();
+    style.backgroundImage = 'url( http://some/url/here1.png  )';
+    expect(style.backgroundImage).toEqual('url(http://some/url/here1.png)');
+  });
+
   test('setting 0 to a padding or margin works', () => {
     var style = new CSSStyleDeclaration();
     style.padding = 0;

diff --git a/lib/parsers.js b/lib/parsers.js
@@ -243,6 +243,7 @@ exports.parseUrl = function parseUrl(val) {
   if (str[0] === '"' || str[0] === "'") {
     str = str.substr(1, str.length - 2);
   }
+  str = str.trim();
 
   var i;
   for (i = 0; i < str.length; i++) {