Skip to content
New issue

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

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

Already on GitHub? # to your account

[PLAY-1115] fix Text input add-on dark mode border and message color #3407

Merged
merged 1 commit into from
May 17, 2024

Conversation

ElisaShapiro
Copy link
Contributor

What does this PR do? A clear and concise description with your runway ticket url.
PLAY-1115 corrects the error color for the border of text input addon cards and the error message in dark mode. These currently display as $error color in both light and dark modes but should change to $error_dark in dark (rails + react).

Screenshots: Screenshots to visualize your addition/change
pre dark mode wrong color for addon and text
post all errors now error_dark

How to test? Steps to confirm the desired behavior:

  1. Go to text input kit page (in either rails or react) and toggle to dark mode (top right corner).
  2. Scroll down to the with error kit example.
  3. See all error borders and text color by the Email Address input in the kit example are $error_dark (the "paler" red).

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.
    - [ ] TESTS I have added test coverage to my code.

@ElisaShapiro ElisaShapiro added bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano labels May 13, 2024
@ElisaShapiro ElisaShapiro self-assigned this May 13, 2024
@ElisaShapiro ElisaShapiro requested review from a team as code owners May 13, 2024 20:52
@co1in-1ewis co1in-1ewis added the Product Approved pending technical review, OK to merge to master label May 15, 2024
@jasperfurniss jasperfurniss added this pull request to the merge queue May 17, 2024
Merged via the queue into master with commit 8504187 May 17, 2024
7 checks passed
@jasperfurniss jasperfurniss deleted the PLAY-1115-fix-Text-input-dark-mode-colors branch May 17, 2024 20:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Fixes to issues discovered in Playbook (USED IN CHANGELOG) milano 20 MAX - Deploy this PR to a review environment via Milano Product Approved pending technical review, OK to merge to master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants