@asteby/metacore-runtime-react 20.1.2 → 20.1.3

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @asteby/metacore-runtime-react
 
+## 20.1.3
+
+### Patch Changes
+
+- 8c6635c: fix(dynamic-table): preserve `view`/`group_by` in the URL when the table syncs its state
+
+  The table's url-sync rebuilt the query string from scratch (only its own
+  page/sort/filter keys) and `replaceState`d it, wiping the route-owned
+  `view`/`group_by` params. On a same-model board↔list pair (e.g. github's Board
+  `?view=kanban` vs Issues `?view=table`) this stripped `?view` on mount, so the
+  URL went bare and the sidebar active-state fell back to the model default —
+  highlighting the wrong sibling until you clicked again. The sync now carries
+  `view`/`group_by` through so the open entry stays highlighted on a single click
+  and `?view` deep-links survive table interaction.
+
 ## 20.1.2
 
 ### Patch Changes

package/dist/dynamic-table.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dynamic-table.d.ts","sourceRoot":"","sources":["../src/dynamic-table.tsx"],"names":[],"mappings":"AAiBA,OAAO,EAKH,KAAK,SAAS,EAajB,MAAM,uBAAuB,CAAA;AAgC9B,OAAO,KAAK,EAAsB,iBAAiB,EAAE,MAAM,wBAAwB,CAAA;AAWnF,MAAM,WAAW,iBAAiB;IAC9B,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,aAAa,CAAC,EAAE,OAAO,CAAA;IACvB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAA;IACxB,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,GAAG,EAAE,GAAG,KAAK,IAAI,CAAA;IAC7C;;;;;OAKG;IACH,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,GAAG,KAAK,IAAI,CAAA;IAC/B,cAAc,CAAC,EAAE,GAAG,CAAA;IACpB,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;IACpC,YAAY,CAAC,EAAE,SAAS,CAAC,GAAG,CAAC,EAAE,CAAA;IAC/B;;;;;OAKG;IACH,iBAAiB,CAAC,EAAE,iBAAiB,CAAA;IACrC;;;;;OAKG;IACH,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB;;;;OAIG;IACH,QAAQ,CAAC,EAAE,MAAM,CAAA;CACpB;AAED,wBAAgB,YAAY,CAAC,EACzB,KAAK,EACL,QAAQ,EACR,aAAoB,EACpB,aAAkB,EAClB,QAAQ,EACR,UAAU,EACV,cAAc,EACd,cAAc,EACd,YAAiB,EACjB,iBAA4C,EAC5C,QAAQ,EACR,QAAQ,GACX,EAAE,iBAAiB,+BA24BnB"}
+{"version":3,"file":"dynamic-table.d.ts","sourceRoot":"","sources":["../src/dynamic-table.tsx"],"names":[],"mappings":"AAiBA,OAAO,EAKH,KAAK,SAAS,EAajB,MAAM,uBAAuB,CAAA;AAgC9B,OAAO,KAAK,EAAsB,iBAAiB,EAAE,MAAM,wBAAwB,CAAA;AAWnF,MAAM,WAAW,iBAAiB;IAC9B,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,aAAa,CAAC,EAAE,OAAO,CAAA;IACvB,aAAa,CAAC,EAAE,MAAM,EAAE,CAAA;IACxB,QAAQ,CAAC,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,GAAG,EAAE,GAAG,KAAK,IAAI,CAAA;IAC7C;;;;;OAKG;IACH,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,GAAG,KAAK,IAAI,CAAA;IAC/B,cAAc,CAAC,EAAE,GAAG,CAAA;IACpB,cAAc,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,CAAA;IACpC,YAAY,CAAC,EAAE,SAAS,CAAC,GAAG,CAAC,EAAE,CAAA;IAC/B;;;;;OAKG;IACH,iBAAiB,CAAC,EAAE,iBAAiB,CAAA;IACrC;;;;;OAKG;IACH,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB;;;;OAIG;IACH,QAAQ,CAAC,EAAE,MAAM,CAAA;CACpB;AAED,wBAAgB,YAAY,CAAC,EACzB,KAAK,EACL,QAAQ,EACR,aAAoB,EACpB,aAAkB,EAClB,QAAQ,EACR,UAAU,EACV,cAAc,EACd,cAAc,EACd,YAAiB,EACjB,iBAA4C,EAC5C,QAAQ,EACR,QAAQ,GACX,EAAE,iBAAiB,+BAw5BnB"}

package/dist/dynamic-table.js

@@ -147,6 +147,20 @@ export function DynamicTable({ model, endpoint, enableUrlSync = true, hiddenColu
         if (!enableUrlSync || !initializedFromUrl.current)
             return;
         const params = new URLSearchParams();
+        // Preserve the route-owned view params. The table rebuilds the query
+        // string from scratch (it only knows its own page/sort/filter keys), but
+        // `view`/`group_by` belong to the renderer choice + the sidebar's
+        // active-state matcher. Carrying them over keeps `?view=table` (vs
+        // `?view=kanban`) in the URL so the open sibling stays highlighted and a
+        // deep-link survives — without this the table wipes `?view` on mount,
+        // the URL goes bare, and the sidebar falls back to the model default
+        // (the "click twice to move the active to the right entry" bug).
+        const current = new URLSearchParams(window.location.search);
+        for (const key of ['view', 'group_by']) {
+            const v = current.get(key);
+            if (v)
+                params.set(key, v);
+        }
         if (pagination.pageIndex > 0)
             params.set('page', String(pagination.pageIndex + 1));
         if (pagination.pageSize !== 10)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@asteby/metacore-runtime-react",
-  "version": "20.1.2",
+  "version": "20.1.3",
   "description": "React runtime for metacore hosts — renders addon contributions dynamically",
   "repository": {
     "type": "git",
@@ -67,8 +67,8 @@
     "typescript": "^6.0.0",
     "vitest": "^4.0.0",
     "zustand": "^5.0.0",
-    "@asteby/metacore-ui": "2.6.2",
-    "@asteby/metacore-sdk": "3.2.0"
+    "@asteby/metacore-sdk": "3.2.0",
+    "@asteby/metacore-ui": "2.6.2"
   },
   "scripts": {
     "build": "tsc -p tsconfig.json",

package/src/dynamic-table.tsx

@@ -257,6 +257,19 @@ export function DynamicTable({
     useEffect(() => {
         if (!enableUrlSync || !initializedFromUrl.current) return
         const params = new URLSearchParams()
+        // Preserve the route-owned view params. The table rebuilds the query
+        // string from scratch (it only knows its own page/sort/filter keys), but
+        // `view`/`group_by` belong to the renderer choice + the sidebar's
+        // active-state matcher. Carrying them over keeps `?view=table` (vs
+        // `?view=kanban`) in the URL so the open sibling stays highlighted and a
+        // deep-link survives — without this the table wipes `?view` on mount,
+        // the URL goes bare, and the sidebar falls back to the model default
+        // (the "click twice to move the active to the right entry" bug).
+        const current = new URLSearchParams(window.location.search)
+        for (const key of ['view', 'group_by']) {
+            const v = current.get(key)
+            if (v) params.set(key, v)
+        }
         if (pagination.pageIndex > 0) params.set('page', String(pagination.pageIndex + 1))
         if (pagination.pageSize !== 10) params.set('per_page', String(pagination.pageSize))
         if (sorting.length > 0) {