Skip to content

Conversation

@georgewrmarshall
Copy link
Contributor

@georgewrmarshall georgewrmarshall commented Oct 24, 2025

Description

  1. Reason for the change: The TextButtonSize enum was not being exported from the root design-system-react-native package, making it inaccessible to consumers. This was reported by a user (Elliot in Slack).
  2. Improvement/Solution: This PR updates packages/design-system-react-native/src/components/index.ts to export TextButtonSize, ensuring it is available when importing from the package's root.

Related issues

Fixes: N/A

Manual testing steps

  1. In a React Native project consuming @metamask/design-system-react-native, attempt to import TextButtonSize:
    import { TextButtonSize } from '@metamask/design-system-react-native';
  2. Verify that TextButtonSize is successfully imported and can be used (e.g., TextButtonSize.Lg).

Screenshots/Recordings

Before

After

Pre-merge author checklist

  • I've followed MetaMask Contributor Docs
  • I've completed the PR template to the best of my ability
  • I’ve included tests if applicable
  • I’ve documented my code using JSDoc format if applicable
  • I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Slack Thread

Open in Cursor Open in Web


Note

Expose TextButtonSize by exporting it from packages/design-system-react-native/src/components/index.ts.

Written by Cursor Bugbot for commit 59e8d5b. This will update automatically on new commits. Configure here.

@cursor
Copy link

cursor bot commented Oct 24, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions
Copy link
Contributor

📖 Storybook Preview

@georgewrmarshall georgewrmarshall changed the title Export TextButtonSize enum from design-system-react-native fix: TextButtonSize enum from design-system-react-native Oct 24, 2025
@georgewrmarshall georgewrmarshall self-assigned this Oct 24, 2025
@georgewrmarshall georgewrmarshall changed the title fix: TextButtonSize enum from design-system-react-native fix: export TextButtonSize enum from design-system-react-native Oct 24, 2025
@georgewrmarshall georgewrmarshall marked this pull request as ready for review October 24, 2025 23:32
@georgewrmarshall georgewrmarshall requested a review from a team as a code owner October 24, 2025 23:32
@georgewrmarshall georgewrmarshall merged commit 2f80a95 into main Oct 24, 2025
42 checks passed
@georgewrmarshall georgewrmarshall deleted the cursor/export-textbuttonsize-enum-from-design-system-react-native-0894 branch October 24, 2025 23:34
@georgewrmarshall georgewrmarshall mentioned this pull request Dec 9, 2025
7 tasks
brianacnguyen pushed a commit that referenced this pull request Dec 9, 2025
## **Description**

This PR releases version 17.0.0 of the MetaMask Design System, featuring
important fixes for mobile font compatibility, the addition of the
ButtonHero component to React, export fixes for React Native, and
various dependency updates including ESLint configuration upgrades.

## **Included PRs**

#836, #837, #838, #840, #843, #845, #846, #847, #848, #850, #851, #852,
#853, #854, #855, #856, #857, #858, #859, #861, #862, #863, #864

## **Manual testing steps**

1. Check package.json version bumps align with included changes
2. Check changelog accurately reflects the release

## **Pre-merge author checklist**

- [x] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs)
- [x] I've completed the PR template to the best of my ability
- [x] I've included tests if applicable
- [x] I've documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [x] I've applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/develop/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

## **Pre-merge reviewer checklist**

- [ ] I've manually tested the PR (e.g. pull and build branch, run the
app, test code being changed).
- [ ] I confirm that this PR addresses all acceptance criteria described
in the ticket it closes and includes the necessary testing evidence such
as recordings and or screenshots.

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Release 17.0.0 introducing React `ButtonHero`, RN font naming changes
and TWRNC preset font family rename (both breaking), plus RN export fix
and dependency updates.
> 
> - **Release 17.0.0**
>   - Bump root `package.json` to `17.0.0`.
> - **React (`@metamask/[email protected]`)**
>   - Add `ButtonHero` component for prominent CTAs.
>   - Update `@metamask/utils` peer to `^11.8.1`.
> - **React Native (`@metamask/[email protected]`)**
> - BREAKING: Rename font files to hyphenated PostScript format for iOS
Metro compatibility.
>   - Export missing `TextButtonSize` enum.
> - Peer: require `@metamask/design-system-twrnc-preset@^0.3.0`; update
`@metamask/utils` peer.
> - **TWRNC preset (`@metamask/[email protected]`)**
> - BREAKING: Rename font family names to hyphenated PostScript format
for iOS/expo-font.
> - **Changelogs**
>   - Update `CHANGELOG.md` links and entries for all packages.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
9a90311. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
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.

4 participants