fix(clayui/css): Fix wrong padding in Clay Picker dropdown

liferay · Jan 18, 2024 · 14e9db8 · 14e9db8
1 parent 51a922b
commit 14e9db8
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 12 deletions.
diff --git a/packages/clay-css/src/scss/atlas/variables/_dropdowns.scss b/packages/clay-css/src/scss/atlas/variables/_dropdowns.scss
@@ -54,7 +54,7 @@ $dropdown-menu: map-deep-merge(
 
 // .dropdown-item
 
-$dropdown-item-padding-x: 1rem !default;
+$dropdown-item-padding-x: 0.5rem !default;
 $dropdown-item-padding-y: 0.34375rem !default;
 
 /// @deprecated as of v3.x use map $dropdown-item instead

diff --git a/packages/clay-css/src/scss/cadmin/components/_dropdowns.scss b/packages/clay-css/src/scss/cadmin/components/_dropdowns.scss
@@ -301,11 +301,6 @@
 	}
 
 	.dropdown-item {
-		padding-right: calc(
-			#{$cadmin-dropdown-item-padding-x} + #{$cadmin-dropdown-item-indicator-size} +
-				#{$cadmin-dropdown-item-indicator-spacer-x}
-		);
-
 		@if ($cadmin-enable-c-inner) {
 			.c-inner {
 				margin-right: calc(

diff --git a/packages/clay-css/src/scss/cadmin/variables/_dropdowns.scss b/packages/clay-css/src/scss/cadmin/variables/_dropdowns.scss
@@ -73,7 +73,7 @@ $cadmin-dropdown-menu: map-deep-merge(
 
 // .dropdown-item
 
-$cadmin-dropdown-item-padding-x: 16px !default;
+$cadmin-dropdown-item-padding-x: 8px !default;
 $cadmin-dropdown-item-padding-y: 8px !default;
 
 $cadmin-dropdown-item-disabled-cursor: $cadmin-disabled-cursor !default;
@@ -396,7 +396,7 @@ $cadmin-dropdown-item-indicator-start: map-deep-merge(
 		display: inline-flex,
 		height: $cadmin-dropdown-item-indicator-size,
 		justify-content: center,
-		left: $cadmin-dropdown-item-padding-x,
+		left: 0.5rem,
 		position: absolute,
 		right: auto,
 		top:

diff --git a/packages/clay-css/src/scss/components/_dropdowns.scss b/packages/clay-css/src/scss/components/_dropdowns.scss
@@ -309,10 +309,6 @@
 	}
 
 	.dropdown-item {
-		padding-right: calc(
-			#{$dropdown-item-padding-x} + #{$dropdown-item-indicator-size} + #{$dropdown-item-indicator-spacer-x}
-		);
-
 		@if ($enable-c-inner) {
 			.c-inner {
 				margin-right: calc(