From d64729530f10a5601c4c15ff393a9fa9153c8b53 Mon Sep 17 00:00:00 2001 From: Tristan Brewster <87919558+xplato@users.noreply.github.com> Date: Tue, 27 Jun 2023 19:19:51 -0600 Subject: [PATCH 1/3] fix: Update `Alert` style to match design spec --- src/lib/ui/Alert/Alert.tsx | 14 ++++++++------ src/styles/_alert.scss | 21 ++++++++++++++++----- 2 files changed, 24 insertions(+), 11 deletions(-) diff --git a/src/lib/ui/Alert/Alert.tsx b/src/lib/ui/Alert/Alert.tsx index 6d9ea54be..16dd22b19 100644 --- a/src/lib/ui/Alert/Alert.tsx +++ b/src/lib/ui/Alert/Alert.tsx @@ -24,12 +24,14 @@ export function Alert(props: AlertProps): JSX.Element { className={classNames('seam-alert', `seam-${variant}-alert`, className)} >
-
- {variant === 'warning' ? ( - - ) : ( - - )} +
+
+ {variant === 'warning' ? ( + + ) : ( + + )} +

{message}

diff --git a/src/styles/_alert.scss b/src/styles/_alert.scss index 5453ca58f..7fcb111dd 100644 --- a/src/styles/_alert.scss +++ b/src/styles/_alert.scss @@ -1,6 +1,7 @@ @use './colors'; @mixin all { + // Alerts .seam-alerts { width: 100%; @@ -13,12 +14,16 @@ &.seam-alerts-space-top { margin-top: 16px; } + + &.seam-alerts-padded { + padding: 0 16px 24px; + } } // Alert .seam-alert { width: 100%; - min-height: 70px; + min-height: 54px; padding: 8px 16px; border-radius: 8px; display: flex; @@ -68,6 +73,11 @@ margin-top: 16px; } + .seam-alert-icon-wrap { + padding: 7px 8px; + padding-left: 0; + } + .seam-alert-icon { width: 24px; height: 24px; @@ -76,12 +86,13 @@ .seam-alert-content { display: flex; justify-content: flex-start; - align-items: center; + align-items: flex-start; flex-direction: row; - gap: 8px; } .seam-alert-message-wrap { + padding: 8px 0; + .seam-alert-message { font-size: 14px; font-weight: 400; @@ -91,7 +102,7 @@ } } - @media only screen and (width <= 500px) { + @media only screen and (width <=500px) { .seam-alert { justify-content: flex-start; flex-direction: column; @@ -112,4 +123,4 @@ } } } -} +} \ No newline at end of file From b4b3429fbdfbf753102754296672f2eb4c838149 Mon Sep 17 00:00:00 2001 From: Seam Bot Date: Wed, 28 Jun 2023 01:21:15 +0000 Subject: [PATCH 2/3] Run format --- src/styles/_alert.scss | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/styles/_alert.scss b/src/styles/_alert.scss index 7fcb111dd..e173131ae 100644 --- a/src/styles/_alert.scss +++ b/src/styles/_alert.scss @@ -1,7 +1,6 @@ @use './colors'; @mixin all { - // Alerts .seam-alerts { width: 100%; @@ -123,4 +122,4 @@ } } } -} \ No newline at end of file +} From c8f5581341c00fc4d2c9b41cbf33c6366a6bcb52 Mon Sep 17 00:00:00 2001 From: Tristan Brewster <87919558+xplato@users.noreply.github.com> Date: Tue, 27 Jun 2023 19:26:50 -0600 Subject: [PATCH 3/3] =?UTF-8?q?fix:=20Remove=20future=20styling=20(?= =?UTF-8?q?=F0=9F=99=88)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- src/styles/_alert.scss | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/styles/_alert.scss b/src/styles/_alert.scss index e173131ae..94905cabc 100644 --- a/src/styles/_alert.scss +++ b/src/styles/_alert.scss @@ -13,10 +13,6 @@ &.seam-alerts-space-top { margin-top: 16px; } - - &.seam-alerts-padded { - padding: 0 16px 24px; - } } // Alert