Bug 454753 - 'Confirm Send Signal' menu UI improvements
Summary: 'Confirm Send Signal' menu UI improvements
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (other bugs)
Version First Reported In: 5.24.5
Platform: Arch Linux Linux
: NOR wishlist
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-02 15:36 UTC by spamless.9v5xj
Modified: 2023-01-11 15:28 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 5.27
Sentry Crash Report:


Attachments
What the bug description says (86.29 KB, image/png)
2022-06-02 15:36 UTC, spamless.9v5xj
Details

Note You need to log in before you can comment on or make changes to this bug.
Description spamless.9v5xj 2022-06-02 15:36:31 UTC
Created attachment 149406 [details]
What the bug description says

SUMMARY
***
The dialog box that pops up asking user to confirm send SIGKILL or SIGTERM is needlessly big, and the button to affirm is ambiguous in meaning
***


STEPS TO REPRODUCE
1. Open System Monitor and switch to the Applications tab.
2. Right-click on an entry within the applications list-table.
3. Hover over 'Send Signal' and choose 'Kill (KILL)' or 'Terminate (TERM)' from the dropdown menu that appears.

OBSERVED RESULT
A dialog box pops up. Not only is the dialog box very large - with about 50% of it being unused whitespace - the button to confirm is labelled 'Quit', which is ambiguous as to whether it means 'dismis and abort action' or 'confirm and go ahead with sending the signal'.

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.24.5
KDE Frameworks Version: 5.94.0
Qt Version: 5.15.4
Comment 1 Ilya Bizyaev 2022-11-19 21:14:52 UTC
On the same topic, applying the "Do not ask again" option cannot be undone — or at least I couldn't find a "Settings" page where such an option could be.
Comment 2 Bug Janitor Service 2023-01-11 14:21:28 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-systemmonitor/-/merge_requests/197
Comment 3 David Redondo 2023-01-11 15:28:15 UTC
Git commit a48a767c30fe62b92b640d4a5f5e83c00f5800b2 by David Redondo.
Committed on 11/01/2023 at 14:21.
Pushed by davidre into branch 'master'.

Be more explicit in kill button texts

Especially for Applications just "Quit" could be confusing if it
quits the Application or quits the dialog.
FIXED-IN:5.27

M  +1    -1    src/faces/applicationstable/contents/ui/FullRepresentation.qml
M  +1    -1    src/faces/processtable/contents/ui/FullRepresentation.qml

https://invent.kde.org/plasma/plasma-systemmonitor/commit/a48a767c30fe62b92b640d4a5f5e83c00f5800b2