-
Notifications
You must be signed in to change notification settings - Fork 327
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
feat: Add highlight for selected groups/rows #1725
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.
Can we also show the checked checkbox if the whole group is selected? It seems it's only displayed on hover currently.
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.
Thanks @marek-mihok! Just a few small changes required.
4cf3e57
to
79291ae
Compare
Thanks @marek-mihok. Could you please include updated screenshots with the new color for multiple themes so that we are sure they look fine beyond the default theme? |
@mturoci I used the "on hover" column header background color to make sure it will look fine for each of the themes. Please check the updated PR description if it is fine for you. |
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.
LGTM, thanks @marek-mihok!
Closes #1713