From 4de60b4bd76ff87db4a584873b88c5f533edf7d3 Mon Sep 17 00:00:00 2001 From: desig9stein Date: Mon, 5 Dec 2022 16:14:29 +0200 Subject: [PATCH 1/2] style(combo, select, input) update the styles for toggle icon --- .../combo/themes/light/combo.material.scss | 4 ---- .../input/themes/light/input.indigo.scss | 2 +- .../select/themes/light/select.indigo.scss | 14 +++++++++----- .../select/themes/light/select.material.scss | 12 ++++++++++++ 4 files changed, 22 insertions(+), 10 deletions(-) diff --git a/src/components/combo/themes/light/combo.material.scss b/src/components/combo/themes/light/combo.material.scss index ab9027d70..379161300 100644 --- a/src/components/combo/themes/light/combo.material.scss +++ b/src/components/combo/themes/light/combo.material.scss @@ -1,10 +1,6 @@ @use '../../../../styles/utilities' as *; @use '../../../select/themes/light/select.material'; -[part='toggle-icon'] { - background: color(gray, 300); -} - igc-input#target:not([outlined]) { &:focus-within { [part='toggle-icon'] { diff --git a/src/components/input/themes/light/input.indigo.scss b/src/components/input/themes/light/input.indigo.scss index f8535a740..8ce44a2a0 100644 --- a/src/components/input/themes/light/input.indigo.scss +++ b/src/components/input/themes/light/input.indigo.scss @@ -6,7 +6,7 @@ $label-focus-color: var(--label-focus-color, color(primary, 500)) !default; display: inline-flex; align-items: center; width: max-content; - height: 100%; + height: calc(100% - #{rem(2px)}); padding-inline: pad-inline(8px, 12px, 16px); } diff --git a/src/components/select/themes/light/select.indigo.scss b/src/components/select/themes/light/select.indigo.scss index cf1f5d7f7..3551a0cd0 100644 --- a/src/components/select/themes/light/select.indigo.scss +++ b/src/components/select/themes/light/select.indigo.scss @@ -1,16 +1,20 @@ -@use '../../../../styles/utilities' as utils; +@use '../../../../styles/utilities' as *; -$label-focus-color: var(--label-focus-color, utils.color(primary, 500)) !default; +$label-focus-color: var(--label-focus-color, color(primary, 500)) !default; + +[part~='toggle-icon'] { + background: color(gray, 300); +} :host { ::slotted([slot='helper-text']) { - color: utils.color(gray, 600); + color: color(gray, 600); } } :host([disabled]) { ::slotted([slot='helper-text']) { - color: utils.color(gray, 300); + color: color(gray, 300); } } @@ -24,7 +28,7 @@ $label-focus-color: var(--label-focus-color, utils.color(primary, 500)) !default :host(:not([invalid]):focus-within) { igc-input[readonly]:not([disabled])::part(container) { - background: utils.color(gray, 100); + background: color(gray, 100); } igc-input[readonly]:not([disabled])::part(label) { diff --git a/src/components/select/themes/light/select.material.scss b/src/components/select/themes/light/select.material.scss index 1ffa4cbb0..d9531d0f6 100644 --- a/src/components/select/themes/light/select.material.scss +++ b/src/components/select/themes/light/select.material.scss @@ -102,3 +102,15 @@ $active-border-width: rem(2px) !default; color: $error-color; } } + +[part~='toggle-icon'] { + background: color(gray, 300); +} + +igc-input:not([outlined]) { + &:focus-within { + [part~='toggle-icon'] { + background: color(gray, 400, .3); + } + } +} From 1bd8bcc825b9afdc233b304fcc805fa1267fb0ea Mon Sep 17 00:00:00 2001 From: Simeon Simeonoff Date: Tue, 6 Dec 2022 09:39:31 +0200 Subject: [PATCH 2/2] theme(combo): remove duplicate code --- src/components/combo/themes/light/combo.material.scss | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/components/combo/themes/light/combo.material.scss b/src/components/combo/themes/light/combo.material.scss index 379161300..014caaf09 100644 --- a/src/components/combo/themes/light/combo.material.scss +++ b/src/components/combo/themes/light/combo.material.scss @@ -1,14 +1,6 @@ @use '../../../../styles/utilities' as *; @use '../../../select/themes/light/select.material'; -igc-input#target:not([outlined]) { - &:focus-within { - [part='toggle-icon'] { - background: color(gray, 400, .3); - } - } -} - :host([outlined]:focus-within) { igc-input::part(suffix) { margin-inline-end: -1px;