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.

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

✨ feat: add antdPrefixCls #143

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

✨ feat: add antdPrefixCls #143

wants to merge 1 commit into from

Conversation

DBvc
Copy link
Contributor

@DBvc DBvc commented Mar 9, 2024

  • Add antdPrefixCls to the return value of useStyles
  • Add antdPrefixCls to CreateStylesUtils

@DBvc
Copy link
Contributor Author

DBvc commented Mar 9, 2024

close #135

@arvinxx
Copy link
Collaborator

arvinxx commented Apr 10, 2024

please add unit test for this code change

@DBvc DBvc force-pushed the feat-antd-prefix branch 2 times, most recently from 9aee4f8 to 076b2a2 Compare June 20, 2024 04:30
@DBvc
Copy link
Contributor Author

DBvc commented Jun 20, 2024

please add unit test for this code change

Sure, I have added the requested tests. Please review the updated PR. Thank you!

@DBvc
✨ feat: add antdPrefixCls
5315496 
- Add antdPrefixCls to the return value of useStyles
- Add antdPrefixCls to CreateStylesUtils
@DBvc
Copy link
Contributor Author

DBvc commented Jun 30, 2024

@arvinxx

@arvinxx
Copy link
Collaborator

arvinxx commented Jul 1, 2024

Sorry, I will review it tomorrow.

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.
2 participants
@DBvc @arvinxx