Skip to content

feat: show skeletons while loading review workspaces - #78

Merged
flexdinesh merged 2 commits into
mainfrom
codex/async-local-startup
Oct 19, 2026
Merged

flexdinesh merged 2 commits into
mainfrom
codex/async-local-startup

Conversation

@flexdinesh

@flexdinesh flexdinesh commented Oct 19, 2026 •

Copy link
Copy Markdown
Owner
  • Fit startup skeletons to the shared Repos/Diff panel and compact header. Preserve saved panel width, view selection, split/unified layout and the disabled Diff action until review data is ready.
  • Reuse diff-frame skeletons during stored-manifest reads without unmounting Pierre. Preserve Sync, Retry and empty states; respect reduced motion and hide decorative placeholders from assistive technology.
  • Wait for final hook delivery before conformance cleanup and allow HTTP shutdown grace plus ownership cleanup, fixing two test lifecycle races.
  • Validation: mise run check:push passed, including 84 Chromium tests and Go race tests; five focused loading/recovery/Sync tests passed. Light/dark desktop and mobile checked; loading WCAG A/AA scans found zero violations, with no mobile overflow or reduced-motion animation.

@flexdinesh
flexdinesh force-pushed the codex/async-local-startup branch from ce2de68 to 7023452 Compare October 19, 2026 07:01
@flexdinesh
flexdinesh marked this pull request as ready for review October 19, 2026 07:01
@flexdinesh
flexdinesh merged commit ae4afd4 into main Oct 19, 2026
2 checks passed
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