Skip to content

Add CI, fix taskbar icon, embed exe icon, 80% zoom#17

Merged
vecnode merged 2 commits into
mainfrom
ci-icon-zoom-updates
Jul 11, 2026
Merged

Add CI, fix taskbar icon, embed exe icon, 80% zoom#17
vecnode merged 2 commits into
mainfrom
ci-icon-zoom-updates

Conversation

@vecnode

@vecnode vecnode commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds .github/workflows/build.yml: builds Release distributables for Windows (MSVC) and Linux (GTK3/WebKit2GTK) on push/PR to main, uploaded as artifacts.
  • Clarifies in README/AGENTS that the app needs no internet at runtime, except when opting into an online p5.js build from the Libraries panel (build-time dependency fetching is the only other internet use).
  • Fixes icons/app_icon_small.ico: it was a solid black 16x16 square (crude 4bpp, no alpha) — regenerated as an anti-aliased black circle at multiple sizes.
  • Adds icons/app.rc, wired into CMakeLists.txt, so app_icon.ico is embedded as neo-processing.exe's own PE resource icon (visible in Explorer/taskbar/Alt+Tab), independent of the runtime window-icon code in main.cpp. No extra step needed in build_and_distribute.bat.
  • public/style.css: html { zoom: 0.8 } so the app starts more compact, giving the editor/preview more effective space in the same window.

Test plan

  • Built locally via build_and_run.bat (Debug, MSVC) — succeeded.
  • Verified icons/app_icon_small.ico renders as an anti-aliased circle at 16px (extracted and inspected the PNG-compressed frame).
  • Verified the compiled neo-processing.exe's embedded PE icon shows the app logo (extracted via Icon.ExtractAssociatedIcon), confirming icons/app.rc is linked correctly.
  • CI itself (Windows + Linux matrix) will be exercised on this PR — not verified locally for Linux (no Linux environment available here).
  • Could not visually confirm the 80% zoom in the live webview window — the app's loopback-only HTTP server isn't reachable from the sandboxed browser pane by design (see AGENTS.md security notes), and desktop screenshot access wasn't available for this unlisted app.

Generated with Claude Code

vecnode and others added 2 commits July 11, 2026 17:49
- .github/workflows/build.yml builds Release distributables for Windows
  (MSVC) and Linux (GTK3/WebKit2GTK) on push/PR, uploaded as artifacts.
- Clarify in README/AGENTS that the app needs no internet at runtime except
  when opting into an online p5.js build from the Libraries panel.
- Regenerate icons/app_icon_small.ico: it was a solid black 16x16 square
  (4bpp, no alpha); it's now an anti-aliased black circle at multiple sizes.
- Add icons/app.rc and wire it into CMakeLists.txt so app_icon.ico is
  embedded as neo-processing.exe's own PE resource icon (Explorer/taskbar),
  independent of the runtime window-icon loading in main.cpp.
- public/style.css: set html { zoom: 0.8 } so the app starts more compact,
  giving the editor/preview more effective space in the same window.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@vecnode vecnode merged commit 9a369ab into main Jul 11, 2026
2 checks passed
@vecnode vecnode deleted the ci-icon-zoom-updates branch July 11, 2026 17:54
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