fix: architecture + UIUX review fixes

1. Pin toggle routes through coordinator + bumps resultVersion for
   SwiftUI reactivity (was mutating class directly, bypassing Equatable)
2. Remove dead onAIFix API from InlineErrorBanner
3. Use NSColor.selectedControlColor for active tab (was wrong
   .selectedContentBackgroundColor.opacity(0.3), broken in dark mode)
4. ResultTabBar height 28→32pt (HIG minimum), replace deprecated
   .cornerRadius with RoundedRectangle, localize context menu strings
5. Consolidate dual ResultSuccessView paths into single branch

Author: datlechin

TablePro/Views/Main/Child/MainEditorContentView.swift

@@ -330,16 +330,16 @@ struct MainEditorContentView: View {
                     )
                 } else if tab.resultColumns.isEmpty && tab.errorMessage == nil
                     && tab.lastExecutedAt != nil && !tab.isExecuting
-                    && !tab.resultSets.isEmpty
                 {
-                    ResultSuccessView(
-                        rowsAffected: tab.rowsAffected,
-                        executionTime: tab.executionTime,
-                        statusMessage: tab.statusMessage
-                    )
-                } else if tab.resultColumns.isEmpty && tab.resultSets.isEmpty {
-                    // All result tabs closed — empty grid
-                    Spacer()
+                    if tab.resultSets.isEmpty {
+                        Spacer()
+                    } else {
+                        ResultSuccessView(
+                            rowsAffected: tab.rowsAffected,
+                            executionTime: tab.executionTime,
+                            statusMessage: tab.statusMessage
+                        )
+                    }
                 } else {
                     // Filter panel (collapsible, above data grid)
                     if filterStateManager.isVisible && tab.tabType == .table {
@@ -381,7 +381,9 @@ struct MainEditorContentView: View {
                 coordinator.closeResultSet(id: id)
             },
             onPin: { id in
-                tab.resultSets.first { $0.id == id }?.isPinned.toggle()
+                guard let tabIdx = coordinator.tabManager.selectedTabIndex else { return }
+                coordinator.tabManager.tabs[tabIdx].resultSets.first { $0.id == id }?.isPinned.toggle()
+                coordinator.tabManager.tabs[tabIdx].resultVersion += 1
             }
         )
     }

TablePro/Views/Results/InlineErrorBanner.swift

@@ -10,7 +10,6 @@ import SwiftUI
 struct InlineErrorBanner: View {
     let message: String
     var onDismiss: (() -> Void)?
-    var onAIFix: (() -> Void)?
 
     var body: some View {
         HStack(spacing: 8) {
@@ -21,11 +20,6 @@ struct InlineErrorBanner: View {
                 .lineLimit(3)
                 .textSelection(.enabled)
             Spacer()
-            if let onAIFix {
-                Button("Fix with AI") { onAIFix() }
-                    .buttonStyle(.bordered)
-                    .controlSize(.small)
-            }
             if let onDismiss {
                 Button { onDismiss() } label: {
                     Image(systemName: "xmark")
@@ -41,16 +35,9 @@ struct InlineErrorBanner: View {
 }
 
 #Preview {
-    VStack {
-        InlineErrorBanner(
-            message: "ERROR 1064 (42000): You have an error in your SQL syntax",
-            onDismiss: {},
-            onAIFix: {}
-        )
-        InlineErrorBanner(
-            message: "Connection refused",
-            onDismiss: {}
-        )
-    }
+    InlineErrorBanner(
+        message: "ERROR 1064 (42000): You have an error in your SQL syntax",
+        onDismiss: {}
+    )
     .frame(width: 600)
 }

TablePro/Views/Results/ResultTabBar.swift

@@ -22,7 +22,7 @@ struct ResultTabBar: View {
                 }
             }
         }
-        .frame(height: 28)
+        .frame(height: 32)
         .background(Color(nsColor: .controlBackgroundColor))
     }
 
@@ -31,7 +31,7 @@ struct ResultTabBar: View {
         return HStack(spacing: 4) {
             if rs.isPinned {
                 Image(systemName: "pin.fill")
-                    .font(.system(size: 8))
+                    .font(.system(size: 9))
                     .foregroundStyle(.secondary)
             }
             Text(rs.label)
@@ -47,22 +47,25 @@ struct ResultTabBar: View {
             }
         }
         .padding(.horizontal, 10)
-        .padding(.vertical, 4)
-        .background(isActive ? Color(nsColor: .selectedContentBackgroundColor).opacity(0.3) : Color.clear)
-        .cornerRadius(4)
+        .padding(.vertical, 6)
+        .background(
+            RoundedRectangle(cornerRadius: 4)
+                .fill(isActive ? Color(nsColor: .selectedControlColor) : Color.clear)
+        )
         .contentShape(Rectangle())
         .onTapGesture { activeResultSetId = rs.id }
         .contextMenu {
-            Button(rs.isPinned ? "Unpin" : "Pin Result") { onPin?(rs.id) }
+            Button(rs.isPinned ? String(localized: "Unpin") : String(localized: "Pin Result")) {
+                onPin?(rs.id)
+            }
             Divider()
-            Button("Close") { onClose?(rs.id) }
+            Button(String(localized: "Close")) { onClose?(rs.id) }
                 .disabled(rs.isPinned)
-            Button("Close Others") {
+            Button(String(localized: "Close Others")) {
                 for other in resultSets where other.id != rs.id && !other.isPinned {
                     onClose?(other.id)
                 }
             }
         }
     }
 }
-