-
-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add SettingsExpander #78
Conversation
试过能跑不? |
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's a problem:
When the SettingsExpander is expanded, the root CornerRadius is gone.
After fixing this then we can merge
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.
The SettingExpander seems right now. However there seems to be some issue that is related to the visual states of nested SettingsCard.
I'll try fixing it tomorrow, and then we can merge the this.
Thanks for ur contribution 😁
No description provided.