Add support for margin and padding jsdom#154

BramMeerten · Jun 26, 2024 · 2d8c96c · 2d8c96c
1 parent 0f99b28
commit 2d8c96c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/lib/CSSStyleDeclaration.test.js b/lib/CSSStyleDeclaration.test.js
@@ -731,7 +731,11 @@ describe('CSSStyleDeclaration', () => {
 
   test('supports calc', () => {
     const style = new CSSStyleDeclaration();
-    style.setProperty('width', 'calc(100% - 100px)');
-    expect(style.getPropertyValue('width')).toEqual('calc(100% - 100px)');
+    ['width', 'height', 'margin', 'margin-top', 'bottom', 'right', 'padding'].forEach(
+      (property) => {
+        style.setProperty(property, 'calc(100% - 100px)');
+        expect(style.getPropertyValue(property)).toEqual('calc(100% - 100px)');
+      }
+    );
   });
 });
diff --git a/lib/properties/margin.js b/lib/properties/margin.js
@@ -12,6 +12,7 @@ var isValid = function (v) {
     type === TYPES.NULL_OR_EMPTY_STR ||
     type === TYPES.LENGTH ||
     type === TYPES.PERCENT ||
+    type === TYPES.CALC ||
     (type === TYPES.INTEGER && (v === '0' || v === 0))
   );
 };

diff --git a/lib/properties/padding.js b/lib/properties/padding.js
@@ -9,6 +9,7 @@ var isValid = function (v) {
     type === TYPES.NULL_OR_EMPTY_STR ||
     type === TYPES.LENGTH ||
     type === TYPES.PERCENT ||
+    type === TYPES.CALC ||
     (type === TYPES.INTEGER && (v === '0' || v === 0))
   );
 };