fix(datagrid): address code review issues

Author: J2TeamNNL

CHANGELOG.md

@@ -46,6 +46,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Opening a `.sql` file names the tab after the file instead of showing "SQL Query". (#1220)
 - 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.
 - 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.
+- 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.45.0] - 2026-05-26
 

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,6 +67,8 @@ extension MainContentView {
                         selectedTab.tableContext.databaseName != session.activeDatabase
                     {
                         await coordinator.switchDatabase(to: selectedTab.tableContext.databaseName)
+                    } else if coordinator.selectedTabHiddenColumns.isEmpty {
+                        coordinator.runQuery()
                     } else {
                         coordinator.lazyLoadCurrentTabIfNeeded()
                     }