Summary: | Removing user results in visually broken UI | ||
---|---|---|---|
Product: | [Applications] systemsettings | Reporter: | Nicolas Fella <nicolas.fella> |
Component: | kcm_users | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | nate, notmart, uhhadd |
Priority: | NOR | ||
Version: | 5.24.0 | ||
Target Milestone: | --- | ||
Platform: | Other | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/commit/3e32e78ece5f416c0d2b38a0951232ce86686a92 | Version Fixed In: | 5.24.2 |
Attachments: | Screenshot |
Description
Nicolas Fella
2022-02-13 23:02:24 UTC
Created attachment 146687 [details]
Screenshot
IIRc those are 2 kirigami pages, one for the sidebar one for the user details must be that eleting an user removes or hides also the page, while it should either go to an empty state or load a different user A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1454 Git commit 0a3c947fa27f472a2d532bc5f65d83d029946c99 by Marco Martin. Committed on 22/02/2022 at 14:16. Pushed by mart into branch 'master'. Always ensure there is an user selected when creating or deleting a user is not enough to pop the old page, but a new user needs to be selected (either the newly created one or the one just on top of the deleted one) so that the kcm doesn't stay half empty with an ui that looks broken M +2 -4 kcms/users/package/contents/ui/CreateUser.qml M +2 -6 kcms/users/package/contents/ui/UserDetailsPage.qml M +22 -1 kcms/users/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/0a3c947fa27f472a2d532bc5f65d83d029946c99 Git commit 3e32e78ece5f416c0d2b38a0951232ce86686a92 by Marco Martin. Committed on 22/02/2022 at 15:13. Pushed by mart into branch 'Plasma/5.24'. Always ensure there is an user selected when creating or deleting a user is not enough to pop the old page, but a new user needs to be selected (either the newly created one or the one just on top of the deleted one) so that the kcm doesn't stay half empty with an ui that looks broken M +2 -4 kcms/users/package/contents/ui/CreateUser.qml M +2 -6 kcms/users/package/contents/ui/UserDetailsPage.qml M +22 -1 kcms/users/package/contents/ui/main.qml https://invent.kde.org/plasma/plasma-workspace/commit/3e32e78ece5f416c0d2b38a0951232ce86686a92 |