Skip to content

feat(dashboard): adds state visibility and repo locking#37

Merged
wgordon17 merged 14 commits intogordon-code:mainfrom
wgordon17:worktree-state-visibility
Mar 30, 2026
Merged

feat(dashboard): adds state visibility and repo locking#37
wgordon17 merged 14 commits intogordon-code:mainfrom
wgordon17:worktree-state-visibility

Conversation

@wgordon17
Copy link
Copy Markdown
Member

Summary

  • Adds hot-poll shimmer/flash animations on item rows with inline peek for collapsed repo headers
  • Adds per-tab repo lock controls with locked-first ordering and reorder highlight animation
  • Includes pre-existing accessibility fixes (role=img, aria-labels, reduced-motion, keyboard focus)

- cor-1: guards isLast with idx !== -1 in RepoLockControls
- cor-2: replaces prevValues.size === 0 with boolean init flag
- cor-3: aggregates multi-item peek updates per repo
- perf-1: replaces indexOf sort with Map-based index lookup
- perf-2: updates prevValues in-place on early-return paths
- cq-1: extracts PeekUpdate interface to shared grouping module
- cq-2: extracts reorder-highlight logic to createReorderHighlight
- cq-4: moves setsEqual to shared collections module
- cq-5: fixes JSX indentation in PullRequestsTab collapsed summary
- qa-1: adds onEnd test for error/throw path in hot-poll coordinator
- qa-2: adds stopPropagation test for unlocked pin button
- qa-5: adds WorkflowSummaryCard hotPollingRunIds/flashingRunIds tests
- qa-6: adds setsEqual unit tests
- Eliminates ~140 lines of duplicated flash detection logic between tabs
- Fixes unbounded prevValues Map leak: full-refresh path prunes stale entries
- Adds tab-level shimmer prop threading tests (qa-3)
- Adds createFlashDetection unit tests for mass-flash gate and init
@wgordon17 wgordon17 force-pushed the worktree-state-visibility branch from 611efa0 to 486a0be Compare March 30, 2026 21:15
@wgordon17 wgordon17 merged commit 08d7fd5 into gordon-code:main Mar 30, 2026
1 check passed
@wgordon17 wgordon17 deleted the worktree-state-visibility branch March 31, 2026 18:07
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.

1 participant