From 2d5c9b9cd8fa9f30fd61b8d84b1e853113079b52 Mon Sep 17 00:00:00 2001 From: Evan Simkowitz Date: Mon, 9 Dec 2024 18:08:08 -0800 Subject: [PATCH 1/4] Add divider between color and icon selectors --- frontend/app/tab/workspaceswitcher.scss | 19 +++++++++---------- 1 file changed, 9 insertions(+), 10 deletions(-) diff --git a/frontend/app/tab/workspaceswitcher.scss b/frontend/app/tab/workspaceswitcher.scss index 880db794f7..d30cc9ef6c 100644 --- a/frontend/app/tab/workspaceswitcher.scss +++ b/frontend/app/tab/workspaceswitcher.scss @@ -74,7 +74,11 @@ } .expandable-menu-item-group { - margin: 0 8px; + padding: 0 8px; + border: 1px solid transparent; + border-radius: 4px; + + --workspace-color: var(--main-bg-color); &:last-child { margin-bottom: 4px; @@ -85,13 +89,6 @@ .expandable-menu-item { margin: 0; } - } - - .expandable-menu-item-group { - border: 1px solid transparent; - border-radius: 4px; - - --workspace-color: var(--main-bg-color); .menu-group-title-wrapper { display: flex; @@ -167,7 +164,9 @@ grid-gap: 18.5px; // Space between items justify-content: center; align-items: center; - margin-top: 5px; + padding-top: 5px; + padding-bottom: 20px; + border-bottom: 1px solid var(--modal-border-color); .color-circle { width: 15px; @@ -204,7 +203,7 @@ grid-row-gap: 13px; // Space between items justify-content: center; align-items: center; - margin-top: 15px; + padding-top: 15px; .icon-item { font-size: 15px; From 09f08dea11976ac8a525031f820e2007e2e59515 Mon Sep 17 00:00:00 2001 From: Evan Simkowitz Date: Mon, 9 Dec 2024 18:19:55 -0800 Subject: [PATCH 2/4] revert padding, add border before delete --- frontend/app/tab/workspaceswitcher.scss | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/frontend/app/tab/workspaceswitcher.scss b/frontend/app/tab/workspaceswitcher.scss index d30cc9ef6c..6b664252d2 100644 --- a/frontend/app/tab/workspaceswitcher.scss +++ b/frontend/app/tab/workspaceswitcher.scss @@ -74,7 +74,7 @@ } .expandable-menu-item-group { - padding: 0 8px; + margin: 0 8px; border: 1px solid transparent; border-radius: 4px; @@ -205,6 +205,9 @@ align-items: center; padding-top: 15px; + padding-bottom: 15px; + border-bottom: 1px solid var(--modal-border-color); + .icon-item { font-size: 15px; color: oklch(from var(--modal-bg-color) calc(l * 1.5) c h); @@ -222,7 +225,7 @@ display: flex; align-items: center; justify-content: center; - margin-top: 10px; + margin-top: 5px; } } From 6ff7682e047bc6a57786252028df4a5bf3a4f37f Mon Sep 17 00:00:00 2001 From: Evan Simkowitz Date: Mon, 9 Dec 2024 18:23:36 -0800 Subject: [PATCH 3/4] remove border between delete and icons --- frontend/app/tab/workspaceswitcher.scss | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/frontend/app/tab/workspaceswitcher.scss b/frontend/app/tab/workspaceswitcher.scss index 6b664252d2..1eb25b9c8c 100644 --- a/frontend/app/tab/workspaceswitcher.scss +++ b/frontend/app/tab/workspaceswitcher.scss @@ -164,8 +164,8 @@ grid-gap: 18.5px; // Space between items justify-content: center; align-items: center; - padding-top: 5px; - padding-bottom: 20px; + margin-top: 5px; + padding-bottom: 15px; border-bottom: 1px solid var(--modal-border-color); .color-circle { @@ -203,10 +203,7 @@ grid-row-gap: 13px; // Space between items justify-content: center; align-items: center; - padding-top: 15px; - - padding-bottom: 15px; - border-bottom: 1px solid var(--modal-border-color); + margin-top: 15px; .icon-item { font-size: 15px; From 22e2ff8773c8038805cf2fb1e72a9cd08e14f048 Mon Sep 17 00:00:00 2001 From: Evan Simkowitz Date: Mon, 9 Dec 2024 18:27:28 -0800 Subject: [PATCH 4/4] add width to left icon --- frontend/app/tab/workspaceswitcher.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/frontend/app/tab/workspaceswitcher.scss b/frontend/app/tab/workspaceswitcher.scss index 1eb25b9c8c..e95ac00b20 100644 --- a/frontend/app/tab/workspaceswitcher.scss +++ b/frontend/app/tab/workspaceswitcher.scss @@ -146,6 +146,7 @@ .left-icon { font-size: 14px; + width: 16px; } }