-
-
Notifications
You must be signed in to change notification settings - Fork 392
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(components): add star rating #425
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.
Looks good already 👍
Can you rename the PR to: feat(components): add star rating
@devCrossNet in the example provided in the issue, there are a fair few variations on the component (#192) like half stars, all the way to 'glowing' stars. Do you have a view on which of these are useful to include? Otherwise could merge this as a v1 & improve later :) Also, was thinking some feedback on star click would be nice to be clear you clicked on a star. |
I think half stars would be good for displaying, not necessarily for selection. Simple use-case one user gives 5 stars, another one gives 3 - display would be 4,5 stars. I like the idea of the visual feedback on click, maybe @einfalles has some ideas for this component, too? |
45b6512
to
657c6f0
Compare
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.
👍
657c6f0
to
a00b537
Compare
What is accomplished by your PR?
Create new star rating component as suggested in: #192
WIP!
Need to add full test coverage & look at greater flexibility/customisability for the component
Is there something controversial in your PR?
please take a moment to tell us about your choices...
Link to the Issue
or a link to a ticket system...
Checklist
New Feature / Bug Fix