From 5bf39fe0082c2ee8193439a3234438136d2e5b3c Mon Sep 17 00:00:00 2001
From: raphael <raphael@cloud13.de>
Date: Tue, 23 Feb 2021 15:04:47 +0100
Subject: [PATCH] only show text background color picker if text tool is active

---
 src/index.html | 9 ++++++++-
 src/js/main.js | 6 ++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/src/index.html b/src/index.html
index c60451c..1e647b6 100644
--- a/src/index.html
+++ b/src/index.html
@@ -134,7 +134,11 @@
                 <button tool="text" title="write text" type="button" class="whiteboard-tool">
                     <i class="fas fa-font"></i>
                 </button>
-                <button title="text background-color">
+                <button
+                    id="textboxBackgroundColorPickerBtn"
+                    style="display: none"
+                    title="text background-color"
+                >
                     <div
                         id="textboxBackgroundColorPicker"
                         style="
@@ -142,6 +146,9 @@
                             height: 23px;
                             border-radius: 3px;
                             border: 1px solid darkgrey;
+                            left: -4px;
+                            top: -2px;
+                            position: relative;
                         "
                         data-color="#f5f587"
                     ></div>
diff --git a/src/js/main.js b/src/js/main.js
index 7572fe7..a9b12d3 100644
--- a/src/js/main.js
+++ b/src/js/main.js
@@ -304,6 +304,12 @@ function initWhiteboard() {
                 } else {
                     $(".activeToolIcon").html($(this).html()); //Set Active icon the same as the button icon
                 }
+
+                if (activeTool == "text") {
+                    $("#textboxBackgroundColorPickerBtn").show();
+                } else {
+                    $("#textboxBackgroundColorPickerBtn").hide();
+                }
             });
 
         // upload image button