From c8d9cf96451d27a0aa0f20e4b776d7a6bdbcef64 Mon Sep 17 00:00:00 2001
From: DiTus <ditus@o2.pl>
Date: Sat, 21 Mar 2026 23:50:14 +0100
Subject: [PATCH] fix: Show Text tab immediately for all line types

---
 index.html             | 30 +++++++++++++++---------------
 js/ui/drawing-tools.js | 11 ++---------
 2 files changed, 17 insertions(+), 24 deletions(-)

diff --git a/index.html b/index.html
index adae0ba..d3273b7 100644
--- a/index.html
+++ b/index.html
@@ -138,21 +138,21 @@
                  </div>
 
                  <!-- Trendline Settings Panel -->
-                 <div id="trendlineSettingsPanel" class="hidden absolute top-14 left-1/2 -translate-x-1/2 bg-[#1a2333] border border-[#2d3a4f] rounded-lg shadow-2xl z-50 w-[280px] p-0 text-sm font-['Inter']">
-                    <!-- Drag Handle -->
-                    <div id="tlPanelHeader" class="h-8 flex items-center justify-between px-3 border-b border-[#2d3a4f] cursor-move bg-[#1f2937] rounded-t-lg">
-                        <span class="text-[10px] uppercase tracking-wider font-bold text-gray-400">Settings</span>
-                        <button class="text-gray-500 hover:text-white" onclick="window.toggleTLSettings(false)">
-                            <span class="material-symbols-outlined text-sm">close</span>
-                        </button>
-                    </div>
-                    
-                    <div class="p-4">
-                        <!-- Tabs -->
-                        <div class="flex border-b border-[#2d3a4f] mb-4">
-                            <button class="flex-1 py-2 text-center text-blue-500 border-b-2 border-blue-500 font-medium" id="tlTabStyle" onclick="window.switchTLTab('style')">Style</button>
-                            <button class="flex-1 py-2 text-center text-gray-400 hover:text-white" id="tlTabText" onclick="window.switchTLTab('text')" style="display: none;">Text</button>
-                        </div>
+                  <div id="trendlineSettingsPanel" class="hidden absolute top-14 left-1/2 -translate-x-1/2 bg-[#1a2333] border border-[#2d3a4f] rounded-lg shadow-2xl z-50 w-[280px] p-0 text-sm font-['Inter']">
+                     <!-- Drag Handle -->
+                     <div id="tlPanelHeader" class="h-8 flex items-center justify-between px-3 border-b border-[#2d3a4f] cursor-move bg-[#1f2937] rounded-t-lg">
+                         <span class="text-[10px] uppercase tracking-wider font-bold text-gray-400">Settings</span>
+                         <button class="text-gray-500 hover:text-white" onclick="window.toggleTLSettings(false)">
+                             <span class="material-symbols-outlined text-sm">close</span>
+                         </button>
+                     </div>
+                     
+                     <div class="p-4">
+                         <!-- Tabs -->
+                         <div class="flex border-b border-[#2d3a4f] mb-4">
+                             <button class="flex-1 py-2 text-center text-blue-500 border-b-2 border-blue-500 font-medium" id="tlTabStyle" onclick="window.switchTLTab('style')">Style</button>
+                             <button class="flex-1 py-2 text-center text-gray-400 hover:text-white" id="tlTabText" onclick="window.switchTLTab('text')">Text</button>
+                         </div>
 
                         <!-- Style Tab Content -->
                         <div id="tlContentStyle">
diff --git a/js/ui/drawing-tools.js b/js/ui/drawing-tools.js
index 6ae7687..ac899bc 100644
--- a/js/ui/drawing-tools.js
+++ b/js/ui/drawing-tools.js
@@ -1201,12 +1201,13 @@ export class DrawingManager {
             if (isLineWithText) {
                 textTab.className = tab === 'text' ? 'flex-1 py-2 text-center text-blue-500 border-b-2 border-blue-500 font-medium' : 'flex-1 py-2 text-center text-gray-400 hover:text-white';
                 textTab.style.display = 'block';
+                document.getElementById('tlContentText').className = tab === 'text' ? 'block' : 'hidden';
             } else {
                 textTab.style.display = 'none';
+                document.getElementById('tlContentText').className = 'hidden';
             }
             
             document.getElementById('tlContentStyle').className = tab === 'style' ? 'block' : 'hidden';
-            document.getElementById('tlContentText').className = tab === 'text' ? 'block' : 'hidden';
             
             const textPicker = document.getElementById('tlTextColorPicker');
             if (textPicker) textPicker.classList.add('hidden');
@@ -1314,14 +1315,6 @@ export class DrawingManager {
         
         if (show) {
             panel.classList.remove('hidden');
-            const isLineWithText = this.selectedDrawing && ['trend_line', 'ray', 'rectangle', 'horizontal_line', 'vertical_line'].includes(this.selectedDrawing.type);
-            if (!isLineWithText) {
-                this.activeTLTab = 'style';
-                document.getElementById('tlTabStyle').className = 'flex-1 py-2 text-center text-blue-500 border-b-2 border-blue-500 font-medium';
-                document.getElementById('tlTabText').className = 'flex-1 py-2 text-center text-gray-400 hover:text-white';
-                document.getElementById('tlContentStyle').className = 'block';
-                document.getElementById('tlContentText').className = 'hidden';
-            }
             this.updateSettingsPanelUI();
         } else {
             panel.classList.add('hidden');