Fix tests and update polyfill (#1098)

stitchesjs · Oct 3, 2022 · deefb08 · deefb08
1 parent 6aa4571
commit deefb08
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 7 deletions.
diff --git a/packages/core/src/convert/toCssRules.js b/packages/core/src/convert/toCssRules.js
@@ -118,9 +118,9 @@ export const toCssRules = (
 						data = (
 							// preserve object-like data
 							isRuleLike ? data
-							// replace specially-marked numeric property values with pixel versions
+							// replace all non-unitless props that are not custom properties with pixel versions
 							: typeof data === 'number'
-								? data && !(camelName in unitlessProps)
+								? data && !(camelName in unitlessProps) && !(name.charCodeAt(0) === 45)
 									? String(data) + 'px'
 								: String(data)
 							// replace tokens with stringified primitive values

diff --git a/packages/core/src/convert/toPolyfilledValue.js b/packages/core/src/convert/toPolyfilledValue.js
@@ -13,7 +13,6 @@ export const toPolyfilledValue = {
 	hyphens: (d) => ({ WebkitHyphens: d, hyphens: d }),
 	maskImage: (d) => ({ WebkitMaskImage: d, maskImage: d }),
 	maskSize: (d) => ({ WebkitMaskSize: d, maskSize: d }),
-	tabSize: (d) => ({ MozTabSize: d, tabSize: d }),
 	textSizeAdjust: (d) => ({ WebkitTextSizeAdjust: d, textSizeAdjust: d }),
 	userSelect: (d) => ({ WebkitUserSelect: d, userSelect: d }),
 

diff --git a/packages/core/tests/universal-autoprefixer.js b/packages/core/tests/universal-autoprefixer.js
@@ -82,7 +82,7 @@ describe('Autoprefixer', () => {
 			},
 		})()
 
-		expect(toString()).toBe('--sxs{--sxs:1 kPCdtQ}@media{x-element{-moz-tab-size:none;tab-size:none}}')
+		expect(toString()).toBe('--sxs{--sxs:1 kPCdtQ}@media{x-element{tab-size:none}}')
 	})
 
 	test('textSizeAdjust', () => {

diff --git a/packages/core/tests/universal-numeric-values.js b/packages/core/tests/universal-numeric-values.js
@@ -75,7 +75,6 @@ describe('Numeric Values', () => {
 					gridColumnSpan: i,
 					gridColumnStart: i,
 					fontWeight: i,
-					lineClamp: i,
 					lineHeight: i,
 					opacity: i,
 					order: i,
@@ -113,12 +112,10 @@ describe('Numeric Values', () => {
 					'grid-column-span:' + i + ';' +
 					'grid-column-start:' + i + ';' +
 					'font-weight:' + i + ';' +
-					'line-clamp:' + i + ';' +
 					'line-height:' + i + ';' +
 					'opacity:' + i + ';' +
 					'order:' + i + ';' +
 					'orphans:' + i + ';' +
-					'-moz-tab-size:' + i + ';' +
 					'tab-size:' + i + ';' +
 					'widows:' + i + ';' +
 					'z-index:' + i + ';' +
-Original file line number
+Diff line change
@@ Expand Up / @@ -82,7 +82,7 @@ describe('Autoprefixer', () => { @@
     			},
     		})()
-    		expect(toString()).toBe('--sxs{--sxs:1 kPCdtQ}@media{x-element{-moz-tab-size:none;tab-size:none}}')
+    		expect(toString()).toBe('--sxs{--sxs:1 kPCdtQ}@media{x-element{tab-size:none}}')
     	})
     	test('textSizeAdjust', () => {
@@ Expand Down @@