-
-
Notifications
You must be signed in to change notification settings - Fork 50.1k
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: migrate less to token for Rate #42135
Conversation
No dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No dependency changes detected in pull request Pull request alert summary
|
| less 变量 | Component Token | 备注 | | ||
| --- | --- | --- | | ||
| `@rate-star-color` | `rateStarColor` | - | | ||
| `@rate-star-bg` | - | 由于样式变化已废弃 | |
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.
这个就是 defaultColor 了,可以吧 defaultColor 改个名
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## feature #42135 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 619 640 +21
Lines 11048 10843 -205
Branches 3081 2945 -136
==========================================
- Hits 11048 10843 -205
☔ View full report in Codecov by Sentry. |
补一个 debug demo 哈 |
[中文版模板 / Chinese template]
🤔 This is a ...
🔗 Related issue link
#41884
💡 Background and solution
📝 Changelog
Rate
组件的 less token☑️ Self-Check before Merge
🚀 Summary
🤖 Generated by Copilot at 52f63a9
Refactored the
Rate
component style hook and updated the migration guide for less variables. This improves the code quality and the user experience of customizing theRate
component.🔍 Walkthrough
🤖 Generated by Copilot at 52f63a9
Rate
component to usegenComponentStyleHook
with default token (link,link)Rate
component token mapping in migration guide for less variables (link,link)