From b5ba015af1ed9e4759302235833b0a9b75b0a833 Mon Sep 17 00:00:00 2001
From: Eugene <eugene@shiptime.com>
Date: Sun, 14 Jun 2026 08:54:03 -0400
Subject: [PATCH] fix: single-click on sidebar table only toggles columns, not
 a new tab (#54)

Single-clicking a table in the schema browser used to both expand/collapse
its column list and open (or switch to) a SELECT query tab, navigating users
away from unsaved work just for wanting to peek at column names.

Single-click now only toggles the column list. Opening a query tab is a
deliberate action: double-click the table, or pick "Query table" from the
right-click context menu (both call the existing onTableSelect path, so the
dedupe-by-(schema,table) and PK-aware template behaviour is unchanged).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 src/components/SchemaBrowser.tsx | 15 +++++++++++++--
 1 file changed, 13 insertions(+), 2 deletions(-)

diff --git a/src/components/SchemaBrowser.tsx b/src/components/SchemaBrowser.tsx
index 0ee16e1..350ddfb 100644
--- a/src/components/SchemaBrowser.tsx
+++ b/src/components/SchemaBrowser.tsx
@@ -250,12 +250,17 @@ export function SchemaBrowser({ schema, activeTable, onTableSelect, onSchemaChan
               <div key={table.name}>
                 <div
                   style={{ ...s.tableRow, ...(activeTable === table.name ? s.tableRowActive : {}) }}
-                  onClick={() => { onTableSelect(table.name); toggleTable(table.name); }}
+                  // Single-click only expands/collapses the column list — it must
+                  // not steal the user's current editor tab. Opening a query tab
+                  // is a deliberate action: double-click, or the context menu's
+                  // "Query table". See #54.
+                  onClick={() => toggleTable(table.name)}
+                  onDoubleClick={() => onTableSelect(table.name)}
                   onContextMenu={(e) => {
                     e.preventDefault();
                     setContextMenu({ x: e.clientX, y: e.clientY, name: table.name, type: 'table' });
                   }}
-                  title={table.comment || undefined}
+                  title={table.comment ? `${table.comment} — double-click to query` : 'Double-click to query'}
                 >
                   <Chevron open={!!expandedTables[table.name]} color={t.textMuted}/>
                   <svg width="12" height="12" viewBox="0 0 24 24" fill="none" stroke={activeTable === table.name ? t.accent : t.textMuted} strokeWidth="2" strokeLinecap="round" strokeLinejoin="round" style={{flexShrink:0}}>
@@ -318,6 +323,12 @@ export function SchemaBrowser({ schema, activeTable, onTableSelect, onSchemaChan
           />
           {contextMenu.type === 'table' && (<>
             <div style={{ height: 1, background: t.borderSubtle, margin: '3px 0' }}/>
+            <CtxItem
+              t={t}
+              onClick={() => { onTableSelect(contextMenu.name); setContextMenu(null); }}
+              icon={<><polyline points="16 18 22 12 16 6"/><polyline points="8 6 2 12 8 18"/></>}
+              label="Query table"
+            />
             {onAlterTable && (
               <CtxItem
                 t={t}