Fix incorrect set value for border-block-width

Scrum · romainmenke · commit 2152c7f222be · 2021-11-16T19:12:52.000+01:00
diff --git a/plugins/postcss-logical/src/lib/transform-border.js b/plugins/postcss-logical/src/lib/transform-border.js
@@ -11,7 +11,7 @@ export default {
 		}),
 		decl.clone({
 			prop: `border-bottom${decl.prop.replace(matchLogicalBorderSide, '$2')}`,
-			value: values[0]
+			value: values[1] || values[0]
 		})
 	],
 
diff --git a/plugins/postcss-logical/test/border.css b/plugins/postcss-logical/test/border.css
@@ -45,6 +45,7 @@ test-flowing-border-width {
 	border-width: inherit;
 	border-inline-end-width: 4px;
 	border-width: inherit;
+	border-block-width: 2px 4px;
 }
 
 test-flowing-border-radius {
diff --git a/plugins/postcss-logical/test/border.expect.css b/plugins/postcss-logical/test/border.expect.css
@@ -93,6 +93,8 @@ test-flowing-border-width {
 	border-width: inherit;
 	border-width: inherit;
 	border-width: inherit;
+	border-top-width: 2px;
+	border-bottom-width: 4px;
 }
 
 test-flowing-border-radius:dir(ltr) {
diff --git a/plugins/postcss-logical/test/border.ltr.expect.css b/plugins/postcss-logical/test/border.ltr.expect.css
@@ -50,6 +50,8 @@ test-flowing-border-width {
 	border-width: inherit;
 	border-right-width: 4px;
 	border-width: inherit;
+	border-top-width: 2px;
+	border-bottom-width: 4px;
 }
 
 test-flowing-border-radius {
diff --git a/plugins/postcss-logical/test/border.preserve.expect.css b/plugins/postcss-logical/test/border.preserve.expect.css
@@ -106,6 +106,9 @@ test-flowing-border-width {
 	border-width: inherit;
 	border-inline-end-width: 4px;
 	border-width: inherit;
+	border-top-width: 2px;
+	border-bottom-width: 4px;
+	border-block-width: 2px 4px;
 }
 
 test-flowing-border-radius:dir(ltr) {

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,7 @@ test-flowing-border-width {`
`45`	`45`	`border-width: inherit;`
`46`	`46`	`border-inline-end-width: 4px;`
`47`	`47`	`border-width: inherit;`
	`48`	`+ border-block-width: 2px 4px;`
`48`	`49`	`}`
`49`	`50`
`50`	`51`	`test-flowing-border-radius {`
Original file line number	Diff line number	Diff line change
`@@ -93,6 +93,8 @@ test-flowing-border-width {`
`93`	`93`	`border-width: inherit;`
`94`	`94`	`border-width: inherit;`
`95`	`95`	`border-width: inherit;`
	`96`	`+ border-top-width: 2px;`
	`97`	`+ border-bottom-width: 4px;`
`96`	`98`	`}`
`97`	`99`
`98`	`100`	`test-flowing-border-radius:dir(ltr) {`
Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,8 @@ test-flowing-border-width {`
`50`	`50`	`border-width: inherit;`
`51`	`51`	`border-right-width: 4px;`
`52`	`52`	`border-width: inherit;`
	`53`	`+ border-top-width: 2px;`
	`54`	`+ border-bottom-width: 4px;`
`53`	`55`	`}`
`54`	`56`
`55`	`57`	`test-flowing-border-radius {`
Original file line number	Diff line number	Diff line change
`@@ -106,6 +106,9 @@ test-flowing-border-width {`
`106`	`106`	`border-width: inherit;`
`107`	`107`	`border-inline-end-width: 4px;`
`108`	`108`	`border-width: inherit;`
	`109`	`+ border-top-width: 2px;`
	`110`	`+ border-bottom-width: 4px;`
	`111`	`+ border-block-width: 2px 4px;`
`109`	`112`	`}`
`110`	`113`
`111`	`114`	`test-flowing-border-radius:dir(ltr) {`