Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Icon Button to use light theme in dark mode #26884

Merged
merged 2 commits into from
Sep 4, 2024

Conversation

garrettbear
Copy link
Contributor

Here’s the reworded PR template for the IconButton component:


Description

This PR updates the IconButton component (also known as ActionButton) to use light theme colors when rendered in dark mode. This update is part of the ongoing brand evolution, ensuring that the visual design is consistent with the new brand guidelines. The primary objective is to align the dark mode appearance of this component with the light theme color palette, improving overall user experience and maintaining visual coherence across the application.

Related issues

Fixes: 735

Manual testing steps

  1. Navigate to any page where the IconButton component is rendered.
  2. Enable dark mode in the application.
  3. Observe the color scheme of the IconButton component to ensure it uses the updated light theme colors.
  4. Compare the colors with the brand guidelines provided.
  5. Test across various instances of the IconButton component in dark mode to ensure consistency.

Screenshots/Recordings

Before

Screenshot 2024-09-03 at 5 04 44 PM Screenshot 2024-09-03 at 5 03 40 PM

After

Screenshot 2024-09-03 at 5 05 26 PM Screenshot 2024-09-03 at 5 03 48 PM

Pre-merge author checklist

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.

This version is tailored specifically for the IconButton component, reflecting the necessary changes and testing steps.

@garrettbear garrettbear added area-UI Relating to the user interface. team-design-system All issues relating to design system in Extension labels Sep 4, 2024
@garrettbear garrettbear requested a review from a team as a code owner September 4, 2024 00:30
Copy link
Contributor

github-actions bot commented Sep 4, 2024

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.15%. Comparing base (1998dc4) to head (1df9736).
Report is 1 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #26884   +/-   ##
========================================
  Coverage    70.15%   70.15%           
========================================
  Files         1425     1425           
  Lines        49653    49653           
  Branches     13893    13893           
========================================
  Hits         34831    34831           
  Misses       14822    14822           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@metamaskbot
Copy link
Collaborator

Builds ready [dca9c34]
Page Load Metrics (1820 ± 115 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint25623631754419201
domContentLoaded157123221802235113
load158023411820239115
domInteractive12103412210
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 21 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

brianacnguyen
brianacnguyen previously approved these changes Sep 4, 2024
Copy link
Contributor

@georgewrmarshall georgewrmarshall left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

  • Checked storybook in light and dark mode ✅
Screenshot 2024-09-04 at 8 11 31 AM Screenshot 2024-09-04 at 8 11 28 AM

Copy link

sonarcloud bot commented Sep 4, 2024

@metamaskbot
Copy link
Collaborator

Builds ready [1df9736]
Page Load Metrics (1893 ± 101 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint43624941824380182
domContentLoaded16642476186219895
load167324931893209101
domInteractive16112472210
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 21 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@garrettbear garrettbear merged commit 1c91c2a into develop Sep 4, 2024
78 checks passed
@garrettbear garrettbear deleted the feat/735/action-buttons-use-light-theme branch September 4, 2024 19:59
@github-actions github-actions bot locked and limited conversation to collaborators Sep 4, 2024
@metamaskbot metamaskbot added the release-12.6.0 Issue or pull request that will be included in release 12.6.0 label Sep 4, 2024
@gauthierpetetin gauthierpetetin added release-12.5.0 Issue or pull request that will be included in release 12.5.0 and removed release-12.6.0 Issue or pull request that will be included in release 12.6.0 labels Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-UI Relating to the user interface. release-12.5.0 Issue or pull request that will be included in release 12.5.0 team-design-system All issues relating to design system in Extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants