-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[structured list] selection variant needs peristent background for selected state #5232
Comments
Seems like this is a larger bug with the actual implementation of the React StructuredList, rather than a simple styling bug. The problem does not exist in vanilla. Can be bundled in with any changes based on the outcome of discussion regarding larger Structured List updates |
@tw15egan Just curious, is there any specific place in code where got an impression that this problem stems from how React StructuredList is implemented? Thanks! |
Just a quick look at it, I'm seeing that it mainly just renders |
Thanks @tw15egan for clarifying! I think we can listen to |
Need to update specs to v11. |
updated the spec! |
For the selected state, the background needs to stay with the checkmark. The background color of the selected state should be
$layer-selected
.Spec:
The text was updated successfully, but these errors were encountered: