-
Notifications
You must be signed in to change notification settings - Fork 225
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
Added Switch Btn to adjust plugin screen alignment #275
Conversation
Added Switch Btn to adjust plugin screen alignment
added switch btn,edit in plugin config page
Hi, apologies for the build failures, allow me to review the failures and refactor the code, I'll update my PR asap, thanks |
Looking forward to your update. |
<Title level={3} type="secondary" style={{margin:0}}>{role}</Title> | ||
</div> | ||
<div style={{ display: 'flex',alignItems:'end',gap:10 }}> | ||
<Switch |
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.
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.
Thanks for the tip.
I can attempt on it, can I get some more idea on it, I didn't quite get an understanding on what you mentioned
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.
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.
Hi, currently the switch can't be toggled ,it can only be changed by clicking edit and changing the status in the modal.
Should I allow changing the status by clicking the switch?
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.
Thank you very much for your contribution. |
You should sync this feature to the admin resource. see : https://github.com/apache/shenyu/tree/master/shenyu-admin/src/main/resources/static Thanks~ |
Implements #274