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

[admin-settings] List of users not sortable in combination with search #8467

Closed
hurradieweltgehtunter opened this issue Feb 17, 2023 · 4 comments · Fixed by #8433
Closed
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@hurradieweltgehtunter
Copy link
Contributor

Steps to reproduce

  1. Open admin settings, panel "Users"
  2. Enter something in searchfield (result should be > 1 users)
  3. Sort table by clicking on column names

Expected behaviour

List of search results gets sorted

Actual behaviour

Sorting arrow in column name switches but list does not get sorted.

Tested in FF & Chrome

https://ocis.ocis-wopi.latest.owncloud.works

Bildschirmaufnahme.2023-02-17.um.11.00.22.mov
@hurradieweltgehtunter hurradieweltgehtunter added the Type:Bug Something isn't working label Feb 17, 2023
@hurradieweltgehtunter hurradieweltgehtunter changed the title [Adminsettings] List of users not sortable in combination with search [admin-settings] List of users not sortable in combination with search Feb 17, 2023
@hurradieweltgehtunter
Copy link
Contributor Author

Just came across the same behaviour in "Groups". But it seems like there is something re-ordered but I cannot figure out, how it gets sorted.

Bildschirmaufnahme.2023-02-17.um.13.34.08.mov

@kulmann kulmann added the Priority:p3-medium Normal priority label Feb 17, 2023
@kulmann
Copy link
Member

kulmann commented Feb 17, 2023

@janackermann could you take a look here?

@AlexAndBear
Copy link
Contributor

@kulmann sure after the sprint tasks, I take over

@AlexAndBear AlexAndBear self-assigned this Feb 17, 2023
@AlexAndBear
Copy link
Contributor

AlexAndBear commented Feb 21, 2023

Will be fixed via #8433
might also improve the performance

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants