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

Dynamically adjust input color based on background in the Select component #719

Merged
merged 16 commits into from
Feb 21, 2024

Conversation

petesiudak
Copy link
Contributor

What

Enhance the Select component to dynamically adjust input color based on the background color.

Testing

  • Is this change covered by the unit tests?

  • Is this change covered by the integration tests?

  • Is this change covered by the automated acceptance tests? (if applicable)

Compatibility

  • Does this change maintain backward compatibility?

Screenshots

Before

After

@petesiudak petesiudak requested a review from a team as a code owner February 19, 2024 12:20
@petesiudak petesiudak changed the title Dynamically adjust Input color based on background in the Select component Dynamically adjust input color based on background in the Select component Feb 19, 2024
Copy link

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-719.du3gff836giqw.amplifyapp.com

@petesiudak petesiudak marked this pull request as draft February 19, 2024 15:05
@petesiudak petesiudak marked this pull request as ready for review February 20, 2024 08:56
@petesiudak petesiudak marked this pull request as draft February 20, 2024 17:28
@petesiudak petesiudak marked this pull request as ready for review February 20, 2024 18:38
@petesiudak petesiudak marked this pull request as draft February 20, 2024 18:38
@petesiudak petesiudak marked this pull request as ready for review February 21, 2024 08:34
raczyk
raczyk previously approved these changes Feb 21, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@petesiudak petesiudak marked this pull request as ready for review February 21, 2024 11:34
@petesiudak petesiudak merged commit 7b992de into master Feb 21, 2024
7 checks passed
@petesiudak petesiudak deleted the ap-3816-select-input-auto-color branch February 21, 2024 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants