This PR addresses an issue where the CSS stylesheet exports were not functioning correctly due to a misconfiguration in the package.json file. Specifically, the exports field in package.json was not correctly configured to include the CSS file, causing the exports to fail.
[x] I've clearly explained what problem this PR is solving and how it is solved.
[x] I've linked related issues
[ ] I've included manual testing steps
[ ] I've included screenshots/recordings if applicable
[x] I’ve included tests if applicable
[x] I’ve documented my code using JSDoc format if applicable
[x] I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.
[x] I’ve properly set the pull request status:
[x] In case it's not yet "ready for review", I've set it to "draft".
[ ] In case it's "ready for review", I've changed it from "draft" to "non-draft".
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.
Description
This PR addresses an issue where the CSS stylesheet exports were not functioning correctly due to a misconfiguration in the
package.json
file. Specifically, theexports
field inpackage.json
was not correctly configured to include the CSS file, causing the exports to fail.Related issues
Fixes: #608 Attempted previous fix: https://github.com/MetaMask/design-tokens/pull/609
Screenshots/Recordings
This PR does not include any visual changes, so screenshots are not applicable.
Before
https://github.com/MetaMask/design-tokens/assets/8112138/70e1fb1c-a2ea-4e2c-a6ca-576f364cc87e
After
https://github.com/MetaMask/design-tokens/assets/8112138/8562d994-d05b-41f9-8e31-4dcd7e3d75ae
Manual testing steps
npm pack
yarn start
in the extensionPre-merge author checklist
Pre-merge reviewer checklist