Merge pull request #7029 from nextcloud-libraries/fix/7005/button-reverse-padding-2

susnux · web-flow · commit db39fe3b190b · 2025-06-10T14:41:25.000+02:00
fix(NcButton): correctly apply reverse padding
diff --git a/src/components/NcButton/NcButton.vue b/src/components/NcButton/NcButton.vue
@@ -804,15 +804,17 @@ function onClick(event: MouseEvent) {
 	&--end &__wrapper {
 		justify-content: end;
 	}
+
 	&--start &__wrapper {
 		justify-content: start;
 	}
+
 	&--reverse &__wrapper {
 		flex-direction: row-reverse;
 	}
 
-	&--reverse#{&}--icon-and-text {
-		padding-inline: var(--button-padding) var(--default-grid-baseline);
+	&--reverse {
+		--button-padding: var(--button-padding-default) var(--default-grid-baseline);
 	}
 
 	&__icon {

Original file line number	Diff line number	Diff line change
`@@ -804,15 +804,17 @@ function onClick(event: MouseEvent) {`
`804`	`804`	`&--end &__wrapper {`
`805`	`805`	`justify-content: end;`
`806`	`806`	`}`
	`807`	`+`
`807`	`808`	`&--start &__wrapper {`
`808`	`809`	`justify-content: start;`
`809`	`810`	`}`
	`811`	`+`
`810`	`812`	`&--reverse &__wrapper {`
`811`	`813`	`flex-direction: row-reverse;`
`812`	`814`	`}`
`813`	`815`
`814`		`- &--reverse#{&}--icon-and-text {`
`815`		`- padding-inline: var(--button-padding) var(--default-grid-baseline);`
	`816`	`+ &--reverse {`
	`817`	`+ --button-padding: var(--button-padding-default) var(--default-grid-baseline);`
`816`	`818`	`}`
`817`	`819`
`818`	`820`	`&__icon {`