Skip to content

feat(views): add grid view with 2D keyboard navigation and view mode persistence (#131) - #154

Merged
Dyu20705 merged 1 commit into
devfrom
issue/131-grid-view
Aug 30, 2026
Merged

feat(views): add grid view with 2D keyboard navigation and view mode persistence (#131)#154
Dyu20705 merged 1 commit into
devfrom
issue/131-grid-view

Conversation

@Dyu20705

Copy link
Copy Markdown
Owner

Issue relationships

Parent issue: #128
Depends on: #130 (PR #153 merged)
Blocks: #132

Problem and scope

Implements Issue #131: Grid View and Custom Reordering.

  • Implements visual Grid View alongside the standard List View for note browsing.
  • Adds #viewModeButton in note panel header actions to toggle between List and Grid modes.
  • Persists user view mode preference in IndexedDB settings store (settings.app.viewMode).
  • Implements 2D keyboard navigation across grid cards (ArrowUp, ArrowDown, ArrowLeft, ArrowRight, Home, End, Enter).
  • Registers notes.toggle-view-mode command bound to Ctrl+Shift+G / Cmd+Shift+G.
  • Preserves virtualized and bounded DOM performance invariants.

Verification evidence

npm run test:content     -> PASS (3/3 passed)
npm run lint             -> PASS (0 errors, 0 warnings)
npm run test:unit        -> PASS (289/289 passed)
npm run test:integration -> PASS (107/107 passed)
npm run test:perf        -> PASS (4/4 passed)
git diff --check         -> PASS (clean)

Copilot AI lite review requested due to automatic review settings August 30, 2026 10:36

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Dyu20705
Dyu20705 merged commit 39c0e19 into dev Aug 30, 2026
1 check failed
@Dyu20705
Dyu20705 deleted the issue/131-grid-view branch August 31, 2026 03:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants