Skip to content

feat: add nitro framework icon#236

Merged
logaretm merged 1 commit intomasterfrom
add-nitro-icon
Apr 27, 2026
Merged

feat: add nitro framework icon#236
logaretm merged 1 commit intomasterfrom
add-nitro-icon

Conversation

@logaretm
Copy link
Copy Markdown
Member

@logaretm logaretm commented Apr 23, 2026

Adds the Nitro framework icon (sourced from https://nitro.build/icon.svg) as javascript-nitro

I made sure to wrap the original 40×40 gradient circle logo in translate + scale so it sits with ~10% padding inside both the 20×20 and 80×80 viewBoxes, to match similar existing icons. I also added a white background rect so the evenodd "N" cutout stays readable on dark UIs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for the Nitro framework by introducing a new nitro SVG icon (20×20 and 80×80 variants) and wiring it into the platform→icon mapping so it can be rendered via PlatformIcon.

Changes:

  • Added nitro.svg icon assets for small (svg/) and large (svg_80x80/) sizes.
  • Updated PLATFORM_TO_ICON to map javascript-nitronitro.
  • Updated src/icons.generated.ts to import and expose the new icon in both icons and iconsLg.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
svg/nitro.svg Adds the 20×20 Nitro icon variant with padding/background for UI legibility.
svg_80x80/nitro.svg Adds the 80×80 Nitro icon variant aligned to the same padding convention.
src/platformIcon.tsx Registers javascript-nitro so callers can request the Nitro icon via platform string.
src/icons.generated.ts Includes the new Nitro SVGs in the generated import + icon maps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@logaretm logaretm merged commit b674400 into master Apr 27, 2026
14 checks passed
@logaretm logaretm deleted the add-nitro-icon branch April 27, 2026 15:44
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.

3 participants