-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
fix(components): [form] not use status-icon and use slot icon style error #16848
base: dev
Are you sure you want to change the base?
Conversation
👋 @warmthsea, thank you for contributing element-plus. |
Hello @warmthsea, thank you for contributing to element-plus, please see our guideline to see how to make contribution |
🧪 Playground Preview: https://element-plus.run/?pr=16848 |
If you change it like this, when setting |
Yes, I'm in a hurry to submit it. It seems that it needs to be modified. |
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.
LGTM
fix: #16846
Before:
use status-icon
not use status-icon
After:
use status-icon
not use status-icon