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

[PLAY-1678] Dark Mode Audit - Gauge #4015

Merged
merged 6 commits into from
Dec 20, 2024
Merged

[PLAY-1678] Dark Mode Audit - Gauge #4015

merged 6 commits into from
Dec 20, 2024

Conversation

Lsimmons98
Copy link
Contributor

@Lsimmons98 Lsimmons98 commented Dec 16, 2024

What does this PR do?

Updates Darkmode styling for the Gauge component

Screenshot 2024-12-18 at 10 56 17 AM
Screenshot 2024-12-18 at 10 55 28 AM

How to test?

  1. Go to 'kits/gauge/react/#complex'
  2. Observe improved darkmode styling

Checklist:

  • LABELS Add a label: enhancement, bug, improvement, new kit, deprecated, or breaking. See Changelog & Labels for details.
  • DEPLOY I have added the milano label to show I'm ready for a review.

@Lsimmons98 Lsimmons98 added milano 20 MAX - Deploy this PR to a review environment via Milano improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) minor Semver Target labels Dec 16, 2024
@Lsimmons98 Lsimmons98 self-assigned this Dec 16, 2024
@Lsimmons98 Lsimmons98 requested a review from a team as a code owner December 16, 2024 20:31
@nidaqg
Copy link
Contributor

nidaqg commented Dec 18, 2024

Dark mode looks great @Lsimmons98 ! Only thing is looks like you're changing the text color on the light mode by accident. Here's your review env:
Screenshot 2024-12-18 at 10 10 45 AM

Here is prod:

Screenshot 2024-12-18 at 10 10 38 AM

@Lsimmons98
Copy link
Contributor Author

Lsimmons98 commented Dec 18, 2024

My bad, should have caught that one. Just pushed up the improved code. @nidaqg

@nidaqg nidaqg added Code Approved Approved by a Playbook Admin and removed Needs Review labels Dec 18, 2024
@nidaqg
Copy link
Contributor

nidaqg commented Dec 18, 2024

My bad, should have caught that one. Just pushed up the improved code. @nidaqg

Looks great! thanks for the quick fix!

@nidaqg nidaqg added Ready for Release merged to master, ready for a versioned released Product Approved pending technical review, OK to merge to master labels Dec 20, 2024
@jasperfurniss jasperfurniss merged commit 12f1d15 into master Dec 20, 2024
12 checks passed
@jasperfurniss jasperfurniss deleted the play-1678 branch December 20, 2024 19:44
Copy link

You merged this pr to master branch:
- Ruby Gem: 14.10.0.pre.rc.22
- NPM: 14.10.0-rc.22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code Approved Approved by a Playbook Admin improvement This is used when your PR contains library upgrades or doc/site improvements. (USED IN CHANGELOG)) milano 20 MAX - Deploy this PR to a review environment via Milano minor Semver Target Product Approved pending technical review, OK to merge to master Ready for Release merged to master, ready for a versioned released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants