Bug 450182 - Removing user results in visually broken UI
Summary: Removing user results in visually broken UI
Status: RESOLVED FIXED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_users (show other bugs)
Version: 5.24.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-02-13 23:02 UTC by Nicolas Fella
Modified: 2022-02-22 15:49 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.24.2
Sentry Crash Report:


Attachments
Screenshot (139.18 KB, image/png)
2022-02-13 23:03 UTC, Nicolas Fella
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nicolas Fella 2022-02-13 23:02:24 UTC
STEPS TO REPRODUCE
1. In Users KCM, create a second user 
2. Open that user
3. Delete the user

OBSERVED RESULT
Right half of the UI disappears, including the header. See screenshot

EXPECTED RESULT
Another user's details are opened, or at least the header fills the full width 

SOFTWARE/OS VERSIONS
KDE Plasma Version: master
KDE Frameworks Version: master
Qt Version: 5.15-kde
Comment 1 Nicolas Fella 2022-02-13 23:03:10 UTC
Created attachment 146687 [details]
Screenshot
Comment 2 Marco Martin 2022-02-14 08:30:54 UTC
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
Comment 3 Bug Janitor Service 2022-02-14 10:04:57 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1454
Comment 4 Marco Martin 2022-02-22 14:17:01 UTC
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
Comment 5 Marco Martin 2022-02-22 15:13:30 UTC
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