🤖 fix: render connection status as input toast #1374
Conversation
Show connection status as an overlay toast anchored to ChatInputSection to avoid layout shifts in creation + workspace views. --- _Generated with `mux` • Model: `openai:gpt-5.2` • Thinking: `xhigh`_
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| > | ||
| <div className={cn("w-full", variant !== "creation" && "mx-auto max-w-4xl")}> | ||
| {/* Toast - show shared toast (slash commands) or variant-specific toast */} | ||
| {!activeToast && <ConnectionStatusToast />} |
There was a problem hiding this comment.
Show connection status even with other chat toasts
Rendering ConnectionStatusToast only when activeToast is null means any existing chat toast (including sticky error toasts from slash-command failures) hides the degraded/reconnecting/lost-connection banner. If the connection drops while an unrelated toast is open, the user will never see the warning or retry button until they manually dismiss that toast, whereas the previous indicator remained visible. This makes connectivity issues invisible during common error flows.
Useful? React with 👍 / 👎.
Render connection status as an overlay toast anchored to ChatInputSection (same placement as the provider-not-configured toast), avoiding layout shift in the creation screen and workspace chat view.
Validation:
Generated with
mux• Model:openai:gpt-5.2• Thinking:xhigh