fix(datagrid): address code review issues

Author: J2TeamNNL

CHANGELOG.md

@@ -7,10 +7,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Fixed
-
-- Changing the editor or data grid font size in Appearance settings now applies immediately and persists across relaunch, instead of resetting and leaving orphan custom themes behind (#1381)
-- Opening a table in a new tab now restores saved hidden columns before the first load, so the initial query matches the visible column set
 ### Added
 
 - Cloudflare Tunnel: connect to a database behind Cloudflare Access by letting TablePro start and stop `cloudflared access tcp` for you, the same way it manages SSH tunnels. Configure it per connection with browser sign-in or a service token. Needs cloudflared installed (`brew install cloudflared`). (#1285)
@@ -29,10 +25,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+- Changing the editor or data grid font size in Appearance settings now applies immediately and persists across relaunch, instead of resetting and leaving orphan custom themes behind (#1381)
+- Opening a table in a new tab now restores saved hidden columns before the first load, so the initial query matches the visible column set
 - Installing or updating a plugin right after updating TablePro now refetches the current plugin list first, so it no longer fails against a stale cached list (the error a restart used to clear). (#1380)
 - Pressing Esc to close the Raw SQL filter suggestions, or to clear a search field, no longer also exits fullscreen. (#1403)
 - Connecting an OAuth-capable MCP client like Claude Code with an invalid or expired token now shows a clear error instead of a confusing "Invalid OAuth error response". (#1409)
 - Data grid row context menus now copy the clicked or focused cell value for Copy, while Copy Rows still keeps the full-row TSV action.
+- The JSON detail popover now shows long string values up to 300 characters in the tree view instead of cutting them off at 80.
 
 ## [0.44.0] - 2026-05-23
 

TablePro/Core/Coordinators/FilterCoordinator.swift

@@ -423,7 +423,7 @@ final class FilterCoordinator {
 
     func clearFilterState() {
         mutateSelectedTabFilterState { state in
-            state.isVisible = false
+            state.isVisible = true
             state.filters = []
             state.appliedFilters = []
         }

TablePro/Models/UI/JSONTreeNode.swift

@@ -57,6 +57,7 @@ internal enum JSONTreeParseError: Error {
 internal enum JSONTreeParser {
     private static let maxNodes = 5_000
     private static let maxInputLength = 100_000
+    private static let maxDisplayLength = 300
 
     static func parse(_ jsonString: String) -> Result<JSONTreeNode, JSONTreeParseError> {
         guard (jsonString as NSString).length <= maxInputLength else {
@@ -108,9 +109,15 @@ internal enum JSONTreeParser {
         case .string(let raw):
             let decoded = JsonSyntaxParser.decodeStringLiteral(raw)
             let escaped = decoded.replacingOccurrences(of: "\"", with: "\\\"")
+            let display: String
+            if (escaped as NSString).length > maxDisplayLength {
+                display = "\"\((escaped as NSString).substring(to: maxDisplayLength))...\""
+            } else {
+                display = "\"\(escaped)\""
+            }
             return JSONTreeNode(
                 key: key, keyPath: keyPath, valueType: .string,
-                displayValue: "\"\(escaped)\"", rawValue: decoded, children: []
+                displayValue: display, rawValue: decoded, children: []
             )
 
         case .number(let raw):

TablePro/Views/Main/Extensions/MainContentView+Setup.swift

@@ -67,8 +67,10 @@ extension MainContentView {
                         selectedTab.tableContext.databaseName != session.activeDatabase
                     {
                         await coordinator.switchDatabase(to: selectedTab.tableContext.databaseName)
+                    } else if coordinator.selectedTabHiddenColumns.isEmpty {
+                        coordinator.runQuery()
                     } else {
-                        coordinator.executeTableTabQueryDirectly()
+                        coordinator.requeryWithColumnScope()
                     }
                 } else {
                     coordinator.needsLazyLoad = true