Skip to content

feat: Add setting to disable radio error toasts (#80) - #86

Merged
cwill747 merged 2 commits into
mainfrom
fix-80
Aug 9, 2026
Merged

feat: Add setting to disable radio error toasts (#80)#86
cwill747 merged 2 commits into
mainfrom
fix-80

Conversation

@cwill747

@cwill747 cwill747 commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Closes #80.

CRC and other radio warnings were shown as toasts at the bottom of the window, covering the message compose box while typing.

Changes

  • New show_radio_error_toasts setting, default off, with a toggle in Settings → Logging. Existing DBs pick up the default since settings persistence is generic over dataclass fields (no migration needed).
  • Radio errors still reach the logs and the header status badge, so nothing is silently dropped.
  • The header badge is now transient. It previously latched on "Radio Error" until the next connect/disconnect, so one CRC warning hid the Connected/Offline state for the rest of the session — with toasts off by default that would have been the only feedback channel. It now reverts to live connection state after 5s, and repeated errors reset the timer rather than stacking timeout sources.
  • The badge's accessible label is updated alongside its text (previously the error path skipped it, so a screen reader still announced "Connection status: Connected").

Testing

uv run pytest — 80 passed. ruff check / ruff format --check clean. New store test covers the default, a persisted False surviving the string round-trip, and True.

🤖 Generated with Claude Code

cwill747 and others added 2 commits August 9, 2026 13:40
CRC and other radio warnings were surfaced as toasts at the bottom of
the window, covering the message compose box during normal use.

Add a `show_radio_error_toasts` setting (default off) with a toggle in
Settings > Logging. Radio errors still reach the logs and the header
status badge.

Make the badge transient while we're here: it previously latched on
"Radio Error" until the next connect/disconnect, so a single CRC
warning hid the Connected/Offline state for the rest of the session.
It now reverts after 5s, and the accessible label is updated alongside
the text so screen readers don't desync.

Closes #80

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@cwill747
cwill747 merged commit 1db24de into main Aug 9, 2026
9 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.

ability to disable error pop up at the bottom of the screen

1 participant