Skip to content

Add configurable fullscreen toggle and tighten CI/settings UX - #1

Merged
dustyghost merged 1 commit into
mainfrom
markj/hide-buttons
Apr 20, 2026
Merged

dustyghost merged 1 commit into
mainfrom
markj/hide-buttons

Conversation

@dustyghost

Copy link
Copy Markdown
Owner

Summary

Cleans up the app’s fullscreen/keybinding flow.
Adds full screen key

Changes

  • make the fullscreen toggle configurable in settings instead of hard-coded
  • keep old settings.toml files working by defaulting missing toggle_full_screen to F
  • prevent duplicate key selections in the settings UI
  • refactor app.rs into smaller input/render helpers
  • add regression tests for settings compatibility and key filtering

Validation

  • cargo test passes locally 11/11

@dustyghost
dustyghost merged commit 148c27e into main Apr 20, 2026
5 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