-
Notifications
You must be signed in to change notification settings - Fork 1
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
Review rating toggle not obvious #189
Comments
This is a good point in the context of the buttons below. What do you think, @neynah @jadeqwang - should we change the styling for the review toggle? |
Okay, @neynah - how does this look? |
Hey @neynah , what do you think of these types of toggle buttons? |
@jadeqwang Since they are like switches, I think they work for values that are opposites. e.g. like / not like, on / off. We could use the toggle button for thumbs up vs thumbs down and then have the appropriate buttons displayed depending on what they toggle. |
I'll mock this up unless you say otherwise. :) |
@neynah go for it. |
Looks great, @neynah thanks. just to be clear on the review bar: is it always going to be two-colour (green and grey)? So if there was 30% Amazing, 30% Gets the job done, 30% I don't like it and 10% It doesn't work, we'd mark it as "60% positive" and have a 60% length green bar with the rest grey? |
@richsilv Ah right. That slipped my mind. I think at the top it should stay as it is. For the bottom, perhaps we can breakdown the distribution somehow. (Let me get back to you) |
@jadeqwang @richsilv Thoughts on this? I figured the review comment box itself probably doesn't need so much space so perhaps we can use some space on the left to show the breakdown/distribution of reviews. I currently have # of reviews per type, but I could also just show the percentage or even both. |
I was surprised to find these buttons above the review box are actually toggles. What makes it extra confusing is that the buttons below look very similar, but are used in a completely different way:
The text was updated successfully, but these errors were encountered: