From cca0cecd0369e104b72a9a70bb0fd9a5193ba21a Mon Sep 17 00:00:00 2001
From: Andre <mail@andrejilderda.nl>
Date: Sun, 12 May 2024 16:56:50 +0200
Subject: [PATCH] fix: duplicate icons in editor toolbar

---
 .changeset/curvy-icons-boil.md              |  5 +++
 src/scss/components/_components.editor.scss | 40 ++++++++++-----------
 2 files changed, 24 insertions(+), 21 deletions(-)
 create mode 100644 .changeset/curvy-icons-boil.md

diff --git a/.changeset/curvy-icons-boil.md b/.changeset/curvy-icons-boil.md
new file mode 100644
index 0000000..321eabb
--- /dev/null
+++ b/.changeset/curvy-icons-boil.md
@@ -0,0 +1,5 @@
+---
+"joplin-plugin-macos-theme": patch
+---
+
+fix: duplicate icons in editor toolbar
diff --git a/src/scss/components/_components.editor.scss b/src/scss/components/_components.editor.scss
index b483e90..0c5eee5 100644
--- a/src/scss/components/_components.editor.scss
+++ b/src/scss/components/_components.editor.scss
@@ -307,14 +307,6 @@
           }
         }
 
-        // icon: back
-        &:nth-of-type(1) span:before {
-          @include icon(chevron-backward);
-        }
-        // icon: forward
-        &:nth-of-type(2) span:before {
-          @include icon(chevron-forward);
-        }
         // icon: external editor
         &:nth-of-type(3):nth-of-type(3) {
           min-width: 2.6rem !important;
@@ -332,6 +324,10 @@
             &::before {
               @include icon(arrow-up-forward-app);
             }
+
+            &:not(:first-child) {
+              display: none;
+            }
           }
         }
       }
@@ -439,32 +435,32 @@
     }
     button:nth-child(3):before {
       content: "H3";
-    } 
+    }
   }
 
   .tox-toolbar__group:nth-child(5) {
     button:nth-child(1):before {
-     @include icon(minus);
-   }
+      @include icon(minus);
+    }
   }
 
   .tox-toolbar__group:nth-child(6) {
     button:nth-child(1):before {
-     @include icon(text-quote);
-   }
+      @include icon(text-quote);
+    }
   }
 
   .tox-toolbar__group:nth-child(7) {
     button:nth-child(1):before {
-     @include icon(tablecells-badge-ellipsis);
-   }
+      @include icon(tablecells-badge-ellipsis);
+    }
   }
-  
+
   .tox-toolbar__group:nth-child(8) {
     button:nth-child(1):before {
-     @include icon(clock);
-   }
- }
+      @include icon(clock);
+    }
+  }
 
   // add divider between editor icons and text input field
   .tox-editor-header {
@@ -632,14 +628,16 @@
     padding: 0;
 
     &:focus-within:hover {
-      &, .tox-split-button {
+      &,
+      .tox-split-button {
         &:not(:hover) {
           @include dropdownItemDefault();
         }
       }
     }
 
-    button, .tox-split-button {
+    button,
+    .tox-split-button {
       @include dropdownItem();
 
       justify-content: flex-start;