Skip to content
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

Various user settings fixes #19533

Merged
merged 8 commits into from
Feb 21, 2020
Merged

Various user settings fixes #19533

merged 8 commits into from
Feb 21, 2020

Conversation

juliusknorr
Copy link
Member

  • Give columns a bit more space and fix overlaying issues
  • Show user/displayname when backend does not offer saving
  • Do not overlay single select boxes over userActions
  • Add tooltip to storage location
  • Properly position popovermenu for user actions

Fixes #19503
Fixes #18920
Fixes #18560
Fixes #19513

image

The grid view is still a pain to get properly here, but i think most urgent issues should be fixed by that PR.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr force-pushed the bugfix/noid/user-settings branch from e05389c to 4e77df7 Compare February 18, 2020 21:48
Copy link
Contributor

@GretaD GretaD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Action menu stays open after you try to delete/wipe/disable a user, but we can fix that with another PR. Lets merge this one.
Everything else looks good. Great work! Thank you Julius.

@ChristophWurst ChristophWurst merged commit b168828 into master Feb 21, 2020
@ChristophWurst ChristophWurst deleted the bugfix/noid/user-settings branch February 21, 2020 10:51
@wiswedel
Copy link
Contributor

Great set of fixes. Can we backport them to 18?

@ankurghai
Copy link

Great set of fixes. Can we backport them to 18?

I think these are for 18.

@juliusknorr
Copy link
Member Author

Can we backport them to 18?

Yes, definitely.

I think these are for 18.

The master branch is already heading towards Nextcloud 19. Fixes for upcoming 18 releases need to be backported to the stable18 branch.

@juliusknorr
Copy link
Member Author

/backport to stable18

@backportbot-nextcloud
Copy link

The backport to stable18 failed. Please do this backport manually.

@kesselb
Copy link
Contributor

kesselb commented Feb 21, 2020

nextcloud test_settings_users (4)

The row is very high with a few users ;)

@rollanders

This comment has been minimized.

@alexanderdd

This comment has been minimized.

@rollanders

This comment has been minimized.

@nextcloud nextcloud locked as resolved and limited conversation to collaborators Mar 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
8 participants