refactor: apply fit-content to mini buttons

sbb-design-systems · Mar 25, 2024 · 1f29663 · 1f29663
1 parent 81582af
commit 1f29663
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 13 deletions.
diff --git a/src/components/core/styles/mixins/buttons.scss b/src/components/core/styles/mixins/buttons.scss
@@ -96,6 +96,7 @@
   #{$host-selector} {
     display: inline-block;
     -webkit-tap-highlight-color: transparent;
+    height: fit-content;
 
     // Use !important here to not interfere with Firefox focus ring definition
     // which appears in normalize css of several frameworks.

diff --git a/src/components/datepicker/datepicker-toggle/datepicker-toggle.scss b/src/components/datepicker/datepicker-toggle/datepicker-toggle.scss
@@ -15,6 +15,9 @@ sbb-popover-trigger {
   --sbb-button-display: flex;
 
   color: var(--sbb-datepicker-control-color);
+
+  // Enables taking full height in order to shift the calendar position to the border bottom of the form field.
+  height: unset;
 }
 
 sbb-icon {

diff --git a/src/components/form-field/form-field/form-field.scss b/src/components/form-field/form-field/form-field.scss
@@ -246,19 +246,6 @@
   margin-block-start: calc((var(--sbb-form-field-min-height) - var(--sbb-size-icon-ui-small)) / 2);
 }
 
-::slotted(
-    :is(
-        sbb-datepicker-next-day,
-        sbb-datepicker-previous-day,
-        sbb-datepicker-toggle,
-        sbb-form-field-clear,
-        sbb-mini-button,
-        sbb-popover-trigger
-      )
-  ) {
-  height: fit-content;
-}
-
 ::slotted(sbb-form-field-clear) {
   :host(:is([data-input-empty], [data-disabled], [data-readonly])) & {
     display: none;