-
Notifications
You must be signed in to change notification settings - Fork 16
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-1724] DARK MODE AUDIT - Map #4126
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good work!
24387cf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work moving the "inline styling" into the SCSS styles!
For future PRs, can you remove the Prettier / formatting edits (like spacing) and only leave the changes related to the PR?
🎉 Congratulations on creating an Alpha Version! Your Alpha for Ruby Gems is 14.11.1.pre.alpha.play1724darkmodeauditmap5409 Your Alpha for NPM is 14.11.1-alpha.play1724darkmodeauditmap5409 |
🎉 Congratulations on creating an Alpha Version! Your Alpha for Ruby Gems is 14.11.1.pre.alpha.play1724darkmodeauditmap5413 Your Alpha for NPM is 14.11.1-alpha.play1724darkmodeauditmap5413 |
🎉 Congratulations on creating an Alpha Version! Your Alpha for Ruby Gems is 14.11.1.pre.alpha.play1724darkmodeauditmap5437 Your Alpha for NPM is 14.11.1-alpha.play1724darkmodeauditmap5437 |
df961ad
We're going with another solution that is less invasive. I'll be closing this PR to open a fresh one with the simplified code. |
What does this PR do? A clear and concise description with your runway ticket url.
PLAY-1724
This PR is a part of the Dark Mode Audit on 'Map'.
We've updated the pop up content, it should display a dark background and white text.
Light mode was also impacted to create this change, the functionality remains the same.
Screenshots: Screenshots to visualize your addition/change
Before (left) and After (right)
How to test? Steps to confirm the desired behavior:
Checklist:
enhancement
,bug
,improvement
,new kit
,deprecated
, orbreaking
. See Changelog & Labels for details.milano
label to show I'm ready for a review.