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(select): add background to selected item and remove color's prop #565

Merged
merged 6 commits into from
Sep 9, 2024

Conversation

felipefialho
Copy link
Contributor

@felipefialho felipefialho commented Sep 6, 2024

What is being delivered?

  • Add a background to the selected item
  • Improve hover color
  • Remove the prop to choose the color of select

What impacts?

Improve colors after selecting an option
closes #561

Evidences

image

image

@felipefialho felipefialho added the bug Something isn't working label Sep 6, 2024
@felipefialho felipefialho self-assigned this Sep 6, 2024
Copy link
Contributor

github-actions bot commented Sep 6, 2024

PR Preview Action v1.4.7
Preview removed because the pull request was closed.
2024-09-09 18:28 UTC

@tassioFront
Copy link
Contributor

question: hey, man. It looks there is a background when there is a selected one. is that right?

Screencast.from.06-09-2024.16.50.14.webm

igorwessel
igorwessel previously approved these changes Sep 6, 2024
@felipefialho felipefialho changed the title fix(select): change selected item background and remove prop to choose component color fix(select): add background to selected item and remove color's prop Sep 9, 2024
@felipefialho felipefialho merged commit 226aa78 into main Sep 9, 2024
4 checks passed
@felipefialho felipefialho deleted the fix/change-select-colors branch September 9, 2024 18:25
This was referenced Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working package/core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix][select] change background color on item selected
5 participants