Skip to content

Conversation

@georgewrmarshall
Copy link
Contributor

@georgewrmarshall georgewrmarshall commented Sep 26, 2025

Description

Updates brand color tokens to align with the latest Figma dark theme specifications.

Note: This PR updates brand color tokens rather than theme-specific tokens to maintain the design system's hierarchical token structure:

Dark Theme Color Updates

Token Value Changes

  • bg-default: #121314 → #131416
  • text-alt: #858B9A → #9B9B9B
  • icon-alt: #858B9A → #9B9B9B
  • icon-muted: #4B505C → #48484E
  • text-muted: #4B505C → #48484E
  • section: #232426 → #1C1D1F
  • subsection: #2A2C32 → #252628

Token Architecture Mapping

semantic-token → brand-color → hex-value

Dark Theme:

  • background-default → grey900 → #131416
  • background-section → grey800 → #1c1d1f
  • background-subsection → grey700 → #252628
  • text-alternative → grey300 → #9b9b9b
  • text-muted → grey600 → #48484e
  • icon-alternative → grey300 → #9b9b9b
  • icon-muted → grey600 → #48484e

Light Theme:

  • text-default → grey900 → #131416
  • icon-default → grey900 → #131416

Related issues

Fixes:

Manual testing steps

  1. Run storybook yarn storybook or go to View PR Storybook and compare against View Main Storybook
  2. Verify brand colors have been updated
  3. Verify dark theme colors are updated:
    • Background default: #131416 (was #121314)
    • Text/icon alternative: #9b9b9b (was #9ca1af)
    • Text/icon muted: #48484e (was #4b505c)
    • Background section: #1c1d1f (was #232426)
    • Background subsection: #252628 (was #2a2c32)
  4. Check light theme still renders correctly with updated text/icon default color
  5. Test Storybook to ensure visual consistency across components

Screenshots/Recordings

Before

Screenshot 2025-09-26 at 8 46 50 AM

After

Screenshot 2025-09-26 at 8 46 31 AM

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.

@github-actions
Copy link
Contributor

📖 Storybook Preview

@georgewrmarshall
Copy link
Contributor Author

@metamaskbot publish-preview

@github-actions
Copy link
Contributor

Preview builds have been published. See these instructions for more information about preview builds.

Expand for full list of packages and versions.
{
  "@metamask-previews/design-system-react": "0.5.0-preview.23d2c11",
  "@metamask-previews/design-system-react-native": "0.4.1-preview.23d2c11",
  "@metamask-previews/design-system-shared": "0.1.1-preview.23d2c11",
  "@metamask-previews/design-system-tailwind-preset": "0.6.1-preview.23d2c11",
  "@metamask-previews/design-system-twrnc-preset": "0.2.1-preview.23d2c11",
  "@metamask-previews/design-tokens": "8.1.1-preview.23d2c11"
}

georgewrmarshall added a commit that referenced this pull request Oct 22, 2025
## **Description**

Adds a new Storybook example story that demonstrates the layered
background token system and text/icon color variations. This story
provides a visual showcase for testing and reviewing the brand color
token updates implemented in PR #839.

The story includes layered background examples (default → section →
subsection), modal-style overlays, and comprehensive text/icon color
demonstrations to validate the design token hierarchy and ensure proper
color contrast relationships.

## **Related issues**

Related to: #839

## **Manual testing steps**

1. Run Storybook: `yarn storybook`
2. Navigate to **"Examples/Backgrounds"** in the sidebar
3. Review the layered background demonstrations:
- **Background Default** → **Background Section** → **Background
Subsection** hierarchy
- **Background Alternative** with **Background Default** overlay (modal
example)
4. Verify text and icon color variations:
   - Default text/icon colors
   - Alternative text/icon colors (updated to #9b9b9b)
   - Muted text/icon colors (updated to #48484e)
5. Compare with main branch Storybook to observe color differences
6. Test in both light and dark themes to ensure proper token application
7. Verify proper color contrast and visual hierarchy

## **Screenshots/Recordings**

### **Before**

No previous version - this is a new story 

### **After**

<img width="1508" height="867" alt="Screenshot 2025-10-22 at 1 37 30 PM"
src="https://github.com/user-attachments/assets/b067066a-9533-4725-9fb7-8fab96b4211d"
/>
<img width="1509" height="864" alt="Screenshot 2025-10-22 at 1 38 05 PM"
src="https://github.com/user-attachments/assets/084d87ce-3d3e-40dd-8c1c-272e52ecaef3"
/>

## **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]
> Adds Backgrounds example stories (web and React Native) demonstrating
layered background tokens and text/icon color variants, and registers
the RN story in Storybook config.
> 
> - **Storybook**:
> - **New examples**: Adds `Examples/Backgrounds` stories in
`apps/storybook-react/stories/Backgrounds.stories.tsx` and
`apps/storybook-react-native/stories/Backgrounds.stories.tsx` showcasing
layered background tokens (`BackgroundDefault → BackgroundSection →
BackgroundSubsection`) with text/icon color variants and button
examples.
> - **React Native config**: Registers the new story in
`apps/storybook-react-native/.storybook/storybook.requires.js`.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
5b50dee. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
@github-actions
Copy link
Contributor

📖 Storybook Preview

@github-actions
Copy link
Contributor

📖 Storybook Preview

@amandaye0h amandaye0h closed this Oct 29, 2025
@amandaye0h amandaye0h deleted the dark-theme-color-update branch October 29, 2025 06:45
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