Skip to content

Commit 80e95d6

Browse files
Copilothotlong
andcommitted
docs: update ROADMAP.md with row click navigation bug fix entry
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent 143b32e commit 80e95d6

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

ROADMAP.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1035,6 +1035,14 @@ The `FlowDesigner` is a canvas-based flow editor that bridges the gap between th
10351035

10361036
**Tests:** Added integration test in `ListViewGroupingPropagation.test.tsx` that verifies toggling a group field via the toolbar immediately updates the rendered schema. All 117 ListView tests pass.
10371037

1038+
### List View Row Click Not Navigating to Record Detail (February 2026)
1039+
1040+
**Root Cause:** `onRowClick` in both `PluginObjectView` (line 772) and `renderListView` (line 599) of `ObjectView.tsx` fell back to `onEdit` / `editHandler`, which only opens an edit form. The `navOverlay.handleClick` from `useNavigationOverlay` — which handles drawer/modal/page navigation modes — was never connected to these click handlers. Additionally, the `useNavigationOverlay` hook was missing the `onNavigate` callback needed for `mode: 'page'` to update the URL.
1041+
1042+
**Fix:** Replaced `onEdit`/`editHandler` fallbacks with `navOverlay.handleClick` in both row click handlers, added `onNavigate` callback to `useNavigationOverlay` that sets the `recordId` URL search parameter, and added `navOverlay` to the `renderListView` useCallback dependency array.
1043+
1044+
**Tests:** All 32 ObjectView tests and 29 useNavigationOverlay tests pass.
1045+
10381046
### ListView Grouping Mode Empty Rows (February 2026)
10391047

10401048
**Root Cause:** When grouping is enabled in list view, `buildGroupTableSchema` in `ObjectGrid.tsx` sets `pagination: false` but inherits `pageSize: 10` from the parent schema. The `DataTableRenderer` filler row logic (`Array.from({ length: Math.max(0, pageSize - paginatedData.length) })`) pads each group table with empty rows up to `pageSize`, creating many blank lines.

0 commit comments

Comments
 (0)