-
-
Notifications
You must be signed in to change notification settings - Fork 4.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
fix(settings): focus lost on selects in users settings #39909
Conversation
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
Signed-off-by: Grigorii K. Shartsev <[email protected]>
should we remove server/apps/settings/src/components/Users/UserRow.vue Lines 385 to 392 in d8d9567
and all other related code for this.idState.xxx
|
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.
It is still used for loading indicators, I only remove its usage from :disable |
Yes... |
Thanks for your first pull request and welcome to the community! Feel free to keep them coming! If you are looking for issues to tackle then have a look at this selection: https://github.com/nextcloud/server/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22 |
Summary
After adding/removing groups the focus is lost from
NcSelect
. You need to catch the select again with tab. This happens because during loading theNcSelect
become disabled and cannot handle focus. To fix - remove disabling on loading state.Also fixed some very minor issues in this component.
🖼️ Screenshots
Todo
NcSelect
s on loading, only keep its loading stateawait
)Drawbacks
Now
NcSelect
is still available during the loading, so the user can add a new group or delete a group. But it should work fine and it is the same as what we have with collaborative tags in the Files sidebar.Checklist