Skip to content

fix: Hush UI polish — compact settings, aligned tray chevrons, window icon - #41

Open
DavidHruby1 wants to merge 2 commits into
mainfrom
fix/hush-ui-polish
Open

DavidHruby1 wants to merge 2 commits into
mainfrom
fix/hush-ui-polish

Conversation

@DavidHruby1

Copy link
Copy Markdown
Owner

Summary

  • Settings dialog: fixed size ~20 % smaller than its natural size (846×737 → 706×590 at the test DPI); pages that do not fit scroll. Import from .env and Reset to Defaults moved to the bottom of the sidebar so the footer no longer forces the width.
  • No more gaps: page content is pinned to the top (QFormLayout used to spread the spare viewport height between rows), and the hidden damaged-vocabulary / damaged-catalog rows now collapse via setRowVisible.
  • Tray menu: submenu chevrons are pulled in 4 px so they end exactly where the separators end.
  • Window icon: apply_theme sets the Hush mark as the application window icon, so the settings / recovery / message box title bars show it instead of the default icon.
  • Includes the pending .gitignore change (ignore local navod.html).

Test plan

  • python -m unittest discover -s tests (298 OK, offscreen)
  • python -m compileall src/
  • Offscreen renders of every settings page and the tray menu (chevron/separator end measured pixel-equal)
  • Check on Windows: title-bar icon, dialog size at your DPI, tray menu chevrons

🤖 Generated with Claude Code

DavidHruby1 and others added 2 commits October 16, 2026 22:09
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Settings dialog is a fixed size ~20% below its natural size; long pages
  scroll. Import from .env / Reset to Defaults move to the sidebar bottom so
  the window can narrow.
- Pages are pinned to the top so QFormLayout no longer spreads spare height
  between rows, and hidden damage-repair rows collapse (setRowVisible).
- Tray submenu chevrons end where the separators end.
- App-wide window icon (Hush mark) instead of the default title bar icon.

Co-Authored-By: Claude Opus 5.5 <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