| Summary: | Pointless confirmation dialog when trying to quit the server from the System Tray icon | ||
|---|---|---|---|
| Product: | [Plasma] KRdp | Reporter: | Nate Graham <nate> |
| Component: | general | Assignee: | Unassigned bugs mailing-list <unassigned-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | ahiemstra |
| Priority: | NOR | Keywords: | usability |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/krdp/-/commit/dd956081c8a3158366cc99410b70e6af2344276b | Version Fixed In: | 6.1.1 |
| Sentry Crash Report: | |||
|
Description
Nate Graham
2024-06-11 15:10:47 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/krdp/-/merge_requests/38 Git commit c6c58fb9059610d67456ca9b52fd8ee80326c4b4 by Akseli Lahtinen. Committed on 13/06/2024 at 15:06. Pushed by akselmo into branch 'master'. SessionController: instead of using default quit action, implement our own This will skip the confirmation dialog popup and also handle the shutdown more gracefully since it will talk to dbus first before exiting. Earlier we were using the default action which was not correct way to do this. Related: bug 488362 M +9 -2 server/SessionController.cpp https://invent.kde.org/plasma/krdp/-/commit/c6c58fb9059610d67456ca9b52fd8ee80326c4b4 Git commit dd956081c8a3158366cc99410b70e6af2344276b by Akseli Lahtinen. Committed on 13/06/2024 at 15:06. Pushed by akselmo into branch 'Plasma/6.1'. SessionController: instead of using default quit action, implement our own This will skip the confirmation dialog popup and also handle the shutdown more gracefully since it will talk to dbus first before exiting. Earlier we were using the default action which was not correct way to do this. Related: bug 488362 (cherry picked from commit c6c58fb9059610d67456ca9b52fd8ee80326c4b4) 159645e5 SessionController: instead of using default quit action, implement our own a56b0b95 Use qcoreapplication quit bb76e487 Use local variable for menu Co-authored-by: Akseli Lahtinen <akselmo@akselmo.dev> M +9 -2 server/SessionController.cpp https://invent.kde.org/plasma/krdp/-/commit/dd956081c8a3158366cc99410b70e6af2344276b |