Add CI, fix taskbar icon, embed exe icon, 80% zoom#17
Merged
Conversation
- .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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/workflows/build.yml: builds Release distributables for Windows (MSVC) and Linux (GTK3/WebKit2GTK) on push/PR tomain, uploaded as artifacts.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.icons/app.rc, wired intoCMakeLists.txt, soapp_icon.icois embedded asneo-processing.exe's own PE resource icon (visible in Explorer/taskbar/Alt+Tab), independent of the runtime window-icon code inmain.cpp. No extra step needed inbuild_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
build_and_run.bat(Debug, MSVC) — succeeded.icons/app_icon_small.icorenders as an anti-aliased circle at 16px (extracted and inspected the PNG-compressed frame).neo-processing.exe's embedded PE icon shows the app logo (extracted viaIcon.ExtractAssociatedIcon), confirmingicons/app.rcis linked correctly.AGENTS.mdsecurity notes), and desktop screenshot access wasn't available for this unlisted app.Generated with Claude Code