Skip to content

Shrink the icon so the Marketplace accepts releases (4.3.11) - #1234

Merged
ddehilster merged 1 commit into
VisualText:masterfrom
ddehilster:fix/marketplace-icon-size
Oct 13, 2026
Merged

ddehilster merged 1 commit into
VisualText:masterfrom
ddehilster:fix/marketplace-icon-size

Conversation

@ddehilster

Copy link
Copy Markdown
Member

Every release since 4.3.4 failed at "Publish to the VS Code
Marketplace": "The provided icon image exceeds the maximum allowed
dimensions of 1024 by 1024 pixels." resources/NLPppLogo.png was
2066 x 2066. 4.3.5 and 4.3.6 hit it with vsce 3.9.2, so it is the
Marketplace's limit, not the vsce 4 bump. GitHub releases 4.3.5-4.3.10
exist; the Marketplace still serves 4.3.4.

The icon is resized to 1024 x 1024 (Lanczos, RGBA kept), and
scripts/check-icon.js reads the PNG header of package.json's icon and
fails above 1024, from npm test and both packaging scripts -- CI and
vsce package never checked this, which is how six releases went
green and unpublished.

Also corrects the 4.3.10 entry: the migration keeps the extension's
own color rules, but scopes it never colored now take the theme's
colors, which changes how @rules, rule concepts, "=" and .dict files
look. Measured by rendering real files before and after.

Verified: npm run vsce-package with vsce 4 builds the .vsix with the 1024 icon; check-icon fails on the old icon and passes on the new; full npm test passes.

🤖 Generated with Claude Code

Every release since 4.3.4 failed at "Publish to the VS Code
Marketplace": "The provided icon image exceeds the maximum allowed
dimensions of 1024 by 1024 pixels." resources/NLPppLogo.png was
2066 x 2066. 4.3.5 and 4.3.6 hit it with vsce 3.9.2, so it is the
Marketplace's limit, not the vsce 4 bump. GitHub releases 4.3.5-4.3.10
exist; the Marketplace still serves 4.3.4.

The icon is resized to 1024 x 1024 (Lanczos, RGBA kept), and
scripts/check-icon.js reads the PNG header of package.json's icon and
fails above 1024, from `npm test` and both packaging scripts -- CI and
`vsce package` never checked this, which is how six releases went
green and unpublished.

Also corrects the 4.3.10 entry: the migration keeps the extension's
own color rules, but scopes it never colored now take the theme's
colors, which changes how @rules, rule concepts, "=" and .dict files
look. Measured by rendering real files before and after.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ddehilster
ddehilster merged commit 87f7436 into VisualText:master Oct 13, 2026
5 checks passed
@ddehilster
ddehilster deleted the fix/marketplace-icon-size branch October 13, 2026 21:52
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