fix: trim empty strings

fixes #45
quicoto · Jul 4, 2023 · 7af333c · 7af333c
1 parent cba4e68
commit 7af333c
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 1 deletion.
diff --git a/src/index.js b/src/index.js
@@ -159,7 +159,11 @@ export default (config = {}) => {
           value = prop.indexOf(`data`) !== -1 ? $ref.getAttribute(prop) : $ref[prop];
         }
 
-        setValueByPath(value, [...propPath], dataModel);
+        setValueByPath(
+          typeof value === `string` ? value.trim() : value,
+          [...propPath],
+          dataModel
+        );
       }
     }
   }

diff --git a/src/tests/index.test.js b/src/tests/index.test.js
@@ -672,4 +672,21 @@ describe(`twoWayDataBinding`, () => {
     expect($input.getAttribute(`data-twowayvalue`)).toEqual(`Germany (DE)`);
     expect($input.value).toEqual(`Spain (ES)`);
   });
+
+  it(`Empty HTML line breaks are trimmed when saved to state`, () => {
+    const {
+      container
+    } = render(
+      `<p data-bind="description">
+
+      </p>`,
+      `data-bind`
+    );
+
+    const proxy = twoWayDataBinding({
+      $context: container
+    });
+
+    expect(proxy.description).toEqual(``);
+  });
 });