From caa052e37eebbb0fa01000014f05be4be76be68f Mon Sep 17 00:00:00 2001
From: Henrik Giesel <hengiesel@gmail.com>
Date: Fri, 16 Apr 2021 18:24:43 +0200
Subject: [PATCH] Make clearer which parts of ButtonGroup css are theming
 related

---
 ts/editor-toolbar/ButtonGroup.svelte | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/ts/editor-toolbar/ButtonGroup.svelte b/ts/editor-toolbar/ButtonGroup.svelte
index 98a6cb760..9e2d09731 100644
--- a/ts/editor-toolbar/ButtonGroup.svelte
+++ b/ts/editor-toolbar/ButtonGroup.svelte
@@ -27,14 +27,13 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
         overflow-y: auto;
 
         padding-inline-start: 0;
-        margin: 0 0 calc(var(--toolbar-size) / 15);
+        margin: 0 0 calc(var(--toolbar-size) / 10);
     }
 
-    .border-group {
-        /* buttons' borders exactly overlap each other */
+    .border-overlap-group {
         :global(button),
         :global(select) {
-            margin-left: -2px;
+            margin-left: -1px;
         }
     }
 
@@ -67,16 +66,16 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
             }
         }
 
-        &:not(:nth-child(1)) {
+        &.gap-item:not(:nth-child(1)) {
             margin-left: 1px;
         }
     }
 </style>
 
-<ul {id} class={className} class:border-group={!nightMode}>
+<ul {id} class={className} class:border-overlap-group={!nightMode}>
     {#each buttons as button}
         {#if !button.hidden}
-            <li>
+            <li class:gap-item={nightMode}>
                 <svelte:component this={button.component} {...filterHidden(button)} />
             </li>
         {/if}