Skip to content

feat: redesign disconnected state as a sticky banner#347

Merged
tomasz-tomczyk merged 3 commits intomainfrom
disconnected-redesign
Apr 24, 2026
Merged

feat: redesign disconnected state as a sticky banner#347
tomasz-tomczyk merged 3 commits intomainfrom
disconnected-redesign

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Replaces the full-screen blocking overlay ("Server stopped" modal) with a slim sticky banner inserted after the header — full width, non-intrusive
  • Banner shows a green "Session complete" pill + single-line message: "Server stopped — your review is now read only. Safe to close this tab."
  • Adds a soft fixed scrim over the diff content to reinforce the read-only terminal state without obscuring the review
  • Introduces --crit-finish-* theme variables across all 4 theme blocks

Review

  • Code review: passed
  • Parity audit: N/A (disconnected state is crit-only; no local server in crit-web)

Test plan

  • Trigger server stop while browser is open — banner appears below header, full width
  • Verify in dark, light, and system themes
  • Verify diff content is still readable through the scrim

🤖 Generated with Claude Code

tomasz-tomczyk and others added 3 commits April 24, 2026 15:32
Replaces the full-screen blocking overlay with a slim amber banner
inserted as a sibling after the header, giving it natural full width
and sticky positioning via dynamic top offset from header.offsetHeight.
Uses --crit-yellow-bg / --crit-yellow tokens for more visible coloring.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replaces plain text banner with a 'Session complete' pill (green
checkmark + label) alongside the status message. Adds a soft fixed
scrim over the page content to reinforce the read-only terminal state.
Introduces --crit-finish-* theme variables for pill and scrim colors
across all four theme blocks.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit f2685fd into main Apr 24, 2026
4 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the disconnected-redesign branch April 24, 2026 15:02
tomasz-tomczyk added a commit that referenced this pull request Apr 24, 2026
Keep the sticky banner but remove the darkened/blurred main body
overlay that was added in #347.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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