-
Notifications
You must be signed in to change notification settings - Fork 56
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
Enhance star ratings with readonly/disabled states #1764
Labels
Comments
13 tasks
17 tasks
12 tasks
15 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Our Star rating component has a disabled version that can be found here: https://boosted.orange.com/docs/forms/checks-radios/#disabled-2.
The associated description begins with "Make star ratings look inactive inside or outside a form [...]".
IMO star rating should be enhanded to propose two separate states:
readonly
would have the same rendering as what we show in https://boosted.orange.com/docs/forms/checks-radios/#disabled-2disabled
would show the orange stars as dark gray starsA readonly state is different than a disabled state.
There's also in this spirit something to check with the readonly state because inputs in Bootstrap/Boosted are focusable in readonly state contrary to disabled state. In readonly, when it's focused, the value could be read by a11y tools as well.
The text was updated successfully, but these errors were encountered: