Skip to content

feat(Rating): add new Rating component #1968

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 22 commits into
base: v2
Choose a base branch
from
Draft

feat(Rating): add new Rating component #1968

wants to merge 22 commits into from

Conversation

J-Michalek
Copy link
Contributor

@J-Michalek J-Michalek commented Jun 1, 2025 •
edited
Loading

Resolves: #1966

Features to implement:

  • disabled
  • clerable
  • hoverable
  • uncontrolled
  • steps (1,5 or 1.25 etc) - this seems a little tricky to implement, might skip it for MVP
  • a11y

@pkg-pr-new pkg.pr.new
Copy link

pkg-pr-new bot commented Jun 1, 2025 •
edited
Loading

Open in StackBlitz

npm i https://pkg.pr.new/reka-ui@1968

commit: d810294

J-Michalek added 21 commits June 1, 2025 09:53
@J-Michalek
feat(Rating): use rowing focus
b646ed0
@J-Michalek
feat(rating-item): handle mouse events
05f18c2
@J-Michalek
feat(rating): implement clearable prop
0040286
@J-Michalek
fix(rating): access disabled value properly
fa2c0c4
@J-Michalek
refactor(rating): cleanup
ab82cdd
@J-Michalek
feat(rating): implement hoverable prop
e6fc713
@J-Michalek
feat(rating): implement step prop
d4f141a
@J-Michalek
refactor(rating): improve handling of steps prop
f276a1a
@J-Michalek
refactor(rating): rename some props
9e7135e
@J-Michalek
test(rating): add more stories
d878d18
@J-Michalek
refactor(rating): use radio group
8c9aeba
@J-Michalek
refactor(rating-item-step): remove default as value
7a59a30
@J-Michalek
Merge branch 'v2' of https://github.com/J-Michalek/reka-ui into feat/…
81efb71 
…rating-component
@J-Michalek
fix(rating): make sure clearable prop works after transition to radio…
6f40dd3 
… group
@J-Michalek
refactor(rating-root): remove unused context props
b665ed4
@J-Michalek
refactor(rating-root): extend RadioGroupRootProps
800759b
@J-Michalek
refactor(rating): remove unused component
df31406
@J-Michalek
refactor(rating): rename RatingItemStep to RatimItemIndicator
afbbc38
@J-Michalek
Merge branch 'v2' of https://github.com/J-Michalek/reka-ui into feat/…
27fb217 
…rating-component
@J-Michalek
refactor(rating-root): omit some props from RadioGroupRoot
7bf7bd3
@J-Michalek
test(Rating): wip
d810294
@J-Michalek
Copy link
Contributor Author

@zernonia Hello, could you have a look at this component once you have time? I still need to work on the docs and tests but I think the component is close to being finished, let me know what you think.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature]: Rating component
1 participant
@J-Michalek