Skip to content

feat: Add support for new IcoMoon app JSON format#356

Merged
vonovak merged 1 commit into
expo:masterfrom
tomekzaw:support-new-icomoon-format
Feb 20, 2026
Merged

feat: Add support for new IcoMoon app JSON format#356
vonovak merged 1 commit into
expo:masterfrom
tomekzaw:support-new-icomoon-format

Conversation

@tomekzaw

Copy link
Copy Markdown
Contributor

This PR adds support for new IcoMoon app JSON output schema in createIconSetFromIcoMoon function while keeping compatibility for the old IcoMoon app JSON output schema.

- Support both old format (icons[].properties) and new format (glyphs[].extras)
- Detect format by checking existence of 'icons' or 'glyphs' in config
- Add fallback error for invalid config
- Font family defaults to 'icomoon' when not in config (new format)

Co-authored-by: Cursor <[email protected]>
@vercel

vercel Bot commented Feb 16, 2026

Copy link
Copy Markdown

@tomekzaw is attempting to deploy a commit to the Expo Team on Vercel.

A member of the Team first needs to authorize it.

@vonovak vonovak self-assigned this Feb 17, 2026
amandeepmittal added a commit to expo/expo that referenced this pull request Feb 17, 2026
# Why

The [new IcoMoon app](https://icomoon.io/new-app) exports a different
JSON format than the [old IcoMoon app](https://icomoon.io/app). The
current `createIconSetFromIcoMoon` function in `@expo/vector-icons` only
supports the old app's output, so users who export from the new app
encounter issues. expo/vector-icons#356 adds
support for the new format. This documentation update clarifies the
situation and points users to that PR until it is released.

# How

Added a note in the `createIconSetFromIcoMoon` section in the docs about
icons.

# Test Plan

Make sure that the `createIconSetFromIcoMoon` section renders correctly.

# Checklist

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [x] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)

---------

Co-authored-by: Aman Mittal <[email protected]>

@vonovak vonovak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

thank you!

@vonovak vonovak merged commit 93e8b2e into expo:master Feb 20, 2026
1 check failed
benjaminkomen pushed a commit to benjaminkomen/expo that referenced this pull request Feb 25, 2026
# Why

The [new IcoMoon app](https://icomoon.io/new-app) exports a different
JSON format than the [old IcoMoon app](https://icomoon.io/app). The
current `createIconSetFromIcoMoon` function in `@expo/vector-icons` only
supports the old app's output, so users who export from the new app
encounter issues. expo/vector-icons#356 adds
support for the new format. This documentation update clarifies the
situation and points users to that PR until it is released.

# How

Added a note in the `createIconSetFromIcoMoon` section in the docs about
icons.

# Test Plan

Make sure that the `createIconSetFromIcoMoon` section renders correctly.

# Checklist

- [ ] I added a `changelog.md` entry and rebuilt the package sources
according to [this short
guide](https://github.com/expo/expo/blob/main/CONTRIBUTING.md#-before-submitting)
- [ ] This diff will work correctly for `npx expo prebuild` & EAS Build
(eg: updated a module plugin).
- [x] Conforms with the [Documentation Writing Style
Guide](https://github.com/expo/expo/blob/main/guides/Expo%20Documentation%20Writing%20Style%20Guide.md)

---------

Co-authored-by: Aman Mittal <[email protected]>
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.

2 participants