Conversation
Replace the rasterized PNG versions of the TexturePacker / SpriteIlluminator / PhysicsEditor icons with the official SVG originals extracted from codeandweb.com. The PNGs had a visible white halo on dark backgrounds caused by raster anti-aliasing of the brand's deliberate outer ring; the vector versions render the ring as a crisp 3px line at any size. Sized at 112×112 to match the original PNG dimensions. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Release preparation for melonJS 19.3.0, updating release metadata and refreshing vendor/tool branding assets used in the repository documentation.
Changes:
- Stamped the
19.3.0CHANGELOG entry date (_unreleased_→_2026-05-08_). - Updated README tool badges to use new CodeAndWeb SVG icons (TexturePacker / SpriteIlluminator / PhysicsEditor).
- Added the corresponding SVG icon assets under
media/icons/.
Reviewed changes
Copilot reviewed 2 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| README.md | Switches CodeAndWeb tool badge images from PNG to local SVG assets. |
| packages/melonjs/CHANGELOG.md | Marks 19.3.0 as released with the release date. |
| media/icons/texturepacker.svg | Adds refreshed TexturePacker SVG icon. |
| media/icons/spriteilluminator.svg | Adds refreshed SpriteIlluminator SVG icon. |
| media/icons/physicseditor.svg | Adds refreshed PhysicsEditor SVG icon. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+127
to
+129
| [](https://www.codeandweb.com/texturepacker) | ||
| [](https://www.codeandweb.com/spriteilluminator) | ||
| [](https://www.codeandweb.com/physicseditor) |
Address Copilot review on #1438. The melonjs package's `files` whitelist (packages/melonjs/package.json) does not include `media/`, so relative `media/icons/*.svg` paths would 404 when the README is rendered from the npm tarball (e.g. on npmjs.com). Switch to `https://github.com/melonjs/melonJS/raw/master/media/icons/*.svg`, matching the convention already used by the banner image at the top of the same file. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
obiot
added a commit
that referenced
this pull request
May 11, 2026
Adds the official Capacitor wordmark next to Cordova in the README's Tools integration row, linking to capacitorjs.com. Source: capacitorjs.com header logo (252x48 PNG with transparent background). Saved locally under media/icons/ so the README renders correctly from the npm tarball — same convention as the codeandweb.com SVGs added in #1438. No vector source exists; Ionic doesn't publish Capacitor as an SVG. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
obiot
added a commit
that referenced
this pull request
May 11, 2026
* docs(readme): add Capacitor to the Tools integration row Adds the official Capacitor wordmark next to Cordova in the README's Tools integration row, linking to capacitorjs.com. Source: capacitorjs.com header logo (252x48 PNG with transparent background). Saved locally under media/icons/ so the README renders correctly from the npm tarball — same convention as the codeandweb.com SVGs added in #1438. No vector source exists; Ionic doesn't publish Capacitor as an SVG. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * ci: nudge — extend main.yml comment to trigger CI on PR #1442 --------- Co-authored-by: Claude Opus 4.7 (1M context) <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
Release prep for melonJS 19.3.0.
_unreleased_→_2026-05-08_).Headlines for 19.3.0
Light2dis now a first-class worldRenderable; lights render inside the camera's post-effect FBO bracket; multi-light works on both Canvas and WebGL.LitQuadBatcherwith up to 8 concurrent lights and configurable ambient.RadialGradientEffectshader; Canvas keeps aWeakMapgradient cache. No per-light textures.restore(). NewClippingexample.setMask(Rect|RoundRect)X/Y swap, WebGL vertex-attribute leak between batchers,gl.useProgramleak aftersetLightUniforms, stale custom shader aftersetBatcher,blitTexturetexture-unit cache desync, scrolling-camera light cutout misalignment, stale Light2d gradient on property change.Light2dis now centered on itspos(anchorPoint = (0.5, 0.5)) — pre-19.3 callers that passed top-left coords need to addradiusX/radiusY(new Light2d(x + r, y + r, r)).Test plan
pnpm vitest run— 2933 tests pass.pnpm dist— clean + test + build + doc, 0 errors.pnpm lint— 0 errors.🤖 Generated with Claude Code