Skip to content
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: added rate component #177

Merged
merged 22 commits into from
Sep 13, 2023
Merged

feat: added rate component #177

merged 22 commits into from
Sep 13, 2023

Conversation

vtrbo
Copy link
Member

@vtrbo vtrbo commented Aug 30, 2023

  • support half
  • support more icons
  • support more colors
  • added rate docs
  • added rate unit test

@netlify
Copy link

netlify bot commented Aug 30, 2023

Deploy Preview for ikun-ui ready!

Name Link
🔨 Latest commit bbfbf61
🔍 Latest deploy log https://app.netlify.com/sites/ikun-ui/deploys/650108fbfa1ef40008666414
😎 Deploy Preview https://deploy-preview-177--ikun-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

components/Rate/package.json Outdated Show resolved Hide resolved
docs/components/KRate.md Outdated Show resolved Hide resolved
components/Rate/src/index.svelte Outdated Show resolved Hide resolved
components/Rate/src/index.svelte Outdated Show resolved Hide resolved
@baiwusanyu-c
Copy link
Member

Can the current implementation add the half-selection function?

@vtrbo
Copy link
Member Author

vtrbo commented Aug 30, 2023

Can the current implementation add the half-selection function?

Let me mark it as a draft first, half-selection should be implemented after adjusting the icon component.

@vtrbo vtrbo marked this pull request as draft August 30, 2023 09:57
@vtrbo vtrbo marked this pull request as ready for review September 8, 2023 07:32
docs/components/KRate.md Show resolved Hide resolved
preset/src/shortcuts/rate.ts Outdated Show resolved Hide resolved
@baiwusanyu-c
Copy link
Member

You can merge it after modification

@baiwusanyu-c baiwusanyu-c merged commit 31d39f5 into main Sep 13, 2023
5 checks passed
@vtrbo vtrbo deleted the feature/rate branch September 13, 2023 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants