-
Notifications
You must be signed in to change notification settings - Fork 115
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: Uncheck group limit in admin settings #3758
Conversation
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
68b2f62
to
13d6a38
Compare
Thanks for looking into this. We used to hide the select if the box was unchecked. Maybe we can bring back this behaviour as well? |
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.
Looks good, have only the same feedback as Julius about hiding the dropdown. :)
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
/backport to stable29 |
/backport to stable28 |
Quick tip: Feel free to already comment with the backport requests when opening the PR, that way the backports will get scheduled once merged. Makes it a bit easier to not miss them :) |
Summary
Previously, trying to uncheck the checkbox would simply re-enable it automatically. This PR fixes this issue. Also adds an
inputLabel
prop to theNcSelect
so it doesn't error anymore (and accessibility reasons :D)uncheck-group-limit-richdouments-fix.mp4
Checklist