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(components): [el-rate] rate add size prop #5102

Merged
merged 3 commits into from Jan 7, 2022
Merged

feat(components): [el-rate] rate add size prop #5102

merged 3 commits into from Jan 7, 2022

Conversation

kooriookami
Copy link
Member

Please make sure these boxes are checked before submitting your PR, thank you!

  • Make sure you follow contributing guide English | (中文 | Español | Français).
  • Make sure you are merging your commits to dev branch.
  • Add some descriptions and refer to relative issues for your PR.

@github-actions
Copy link

github-actions bot commented Jan 1, 2022

@kooriookami kooriookami requested a review from a team January 1, 2022 14:48
@@ -12,6 +12,15 @@ describe('Rate.vue', () => {
expect(stars.length).toEqual(10)
})

test('size', () => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add a test case for el-rate without a size prop?

Copy link
Collaborator

@HerringtonDarkholme HerringtonDarkholme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some comments.

Copy link
Member

@jw-foss jw-foss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have left 2 suggestions for you:

  1. Update the document
  2. Removing the checking for size.

packages/components/rate/src/rate.vue Outdated Show resolved Hide resolved
docs/en-US/component/rate.md Outdated Show resolved Hide resolved
Copy link
Member

@jw-foss jw-foss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks. /cc @HerringtonDarkholme

@jw-foss jw-foss merged commit 9e53873 into dev Jan 7, 2022
@jw-foss jw-foss deleted the feat/rate-size branch January 7, 2022 07:43
@element-bot element-bot mentioned this pull request Jan 8, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants