Skip to content

Feature/app-icon#10

Merged
MacBuchi merged 5 commits into
Mainfrom
feature/app-icon
Mar 6, 2026
Merged

Feature/app-icon#10
MacBuchi merged 5 commits into
Mainfrom
feature/app-icon

Conversation

@MacBuchi

@MacBuchi MacBuchi commented Mar 6, 2026

Copy link
Copy Markdown
Owner

Changes

UI — modern header

  • Fixed-height dark header bar matching the BG_Header gradient palette (#090c15) sits above the existing toolbar
  • Logo.png displayed at 76 x 53 px on the left side
  • App title bold, light-blue tint (#daeef8)
  • Subtitle muted cyan (#4a7a92)
  • 2 px accent line (#1e5f7a) between header and toolbar for visual separation
  • Window default size bumped 1060x560 to 1060x628, minsize 900x420 to 900x490

Asset loading (_BUNDLE_DIR / _load_ctk_image)

  • _BUNDLE_DIR resolves to sys._MEIPASS inside a frozen .app, and to the repo root when running from source
  • _load_ctk_image(filename, size) opens a PNG from Pics/ as a CTkImage, returning None gracefully on failure

PyInstaller spec

  • Pics/*.png are now collected into the bundle so the header logo is available in .app / Windows build

Tests — tests/test_app_smoke.py

  • test_app_starts_and_stays_alive: launches MultiChannelWavMixer.py as a subprocess, asserts it is still alive after 6 seconds, then terminates it
  • On failure, stderr is captured and surfaced in the pytest output
  • Skipped on Linux (no native display) — the pytest CI job runs on macos-latest so it always executes there

MacBuchi added 5 commits March 6, 2026 01:22
- Add dark header strip (BG matches BG_Header palette #090c15) with
  Logo.png (76x53), bold app title and muted subtitle
- 2px accent line separating header from toolbar
- Window default height 560→628, minsize 420→490
- _BUNDLE_DIR / _load_ctk_image() helper for frozen + source paths
- Bundle Pics/*.png into PyInstaller spec
- Add tests/test_app_smoke.py: subprocess smoke test verifying the app
  stays alive for 6 s; skipped on Linux (no GUI)
- Pics/AppIcon.icns — full iconset (16–1024 px) generated from App_Logo.png
- Pics/AppIcon.ico  — multi-size (16–256 px) generated from App_Logo.png
- Wire icons into PyInstaller spec:
    BUNDLE icon=Pics/AppIcon.icns (macOS .app)
    EXE    icon=Pics/AppIcon.ico  (Windows .exe)
@MacBuchi
MacBuchi merged commit 31ea542 into Main Mar 6, 2026
6 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