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

Fix wrong color in text inputs #4183

Merged
merged 1 commit into from Oct 13, 2023
Merged

Fix wrong color in text inputs #4183

merged 1 commit into from Oct 13, 2023

Conversation

ivarnakken
Copy link
Member

Description

Also fix wrong styling on prefix icon given by the recent changes to the Icon component.

Result

Before After
image image

Testing

  • I have thoroughly tested my changes.

  • Text inputs now have a correct text color.

  • Tested with and without a prefix, and centered inputs.

  • prefix still works as expected, by focusing on the input when clicked.


Resolves ABA-602

@ivarnakken ivarnakken added review-needed Pull requests that need review bug-fix Pull requests that fix a bug labels Oct 12, 2023
@ivarnakken ivarnakken requested a review from a team October 12, 2023 19:52
@ivarnakken ivarnakken self-assigned this Oct 12, 2023
@linear
Copy link

linear bot commented Oct 12, 2023

ABA-602 Wrong color on text inputs

They are now always var(--color-gray-5). Kind of related to the prefix icon's funky styling.

Copy link
Member

@eikhr eikhr left a comment

Choose a reason for hiding this comment

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

Nice, this was super annoying!

@ivarnakken ivarnakken added the approved Pull requests that have been approved label Oct 13, 2023
Also fix wrong styling on prefix icon given by the recent changes to the
Icon component.
@ivarnakken ivarnakken merged commit 76f082c into master Oct 13, 2023
4 checks passed
@ivarnakken ivarnakken deleted the input-field-color branch October 13, 2023 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Pull requests that have been approved bug-fix Pull requests that fix a bug review-needed Pull requests that need review
Projects
None yet
2 participants