Summary: | Double click to select word | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | xcojack <xcojack> |
Component: | Notifications | Assignee: | Kai Uwe Broulik <kde> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | kdelibs-bugs, nate, nicolas.fella, plasma-bugs |
Priority: | NOR | Keywords: | usability |
Version: | 5.19.5 | ||
Target Milestone: | 1.0 | ||
Platform: | Kubuntu | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-workspace/commit/97022f2f67c269cec2660ae4690e527cf452b322 | Version Fixed In: | 5.22.0 |
Description
xcojack@gmail.com
2021-01-10 15:58:33 UTC
Confirmed. I find myself very slightly annoyed by this when I want to select and copy a one-time code from a website doing 2-factor authentication. The question is whether we want to "jeopardize" the activation of a notification, i.e. if you click, wait for a potential double click (or triple click) before activating it. Unless, we only care about non-activatable notifications, like the ones synced from KDE Connect. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/602 Git commit 97022f2f67c269cec2660ae4690e527cf452b322 by Kai Uwe Broulik. Committed on 30/01/2021 at 17:50. Pushed by broulik into branch 'master'. [Notifications] Don't steal mouse clicks from TextEdit in SelectableLabel Ensures built-in functionality like double- and triple-clicking text works. Also lets us get rid of the custom text selection handling, replacing it with an even filter class. FIXED-IN: 5.22.0 M +1 -0 applets/notifications/CMakeLists.txt M +2 -0 applets/notifications/notificationapplet.cpp M +1 -1 applets/notifications/package/contents/ui/NotificationItem.qml M +2 -1 applets/notifications/package/contents/ui/NotificationPopup.qml M +22 -54 applets/notifications/package/contents/ui/SelectableLabel.qml A +70 -0 applets/notifications/texteditclickhandler.cpp [License: LGPL (v2.1+)] A +57 -0 applets/notifications/texteditclickhandler.h [License: LGPL (v2.1+)] https://invent.kde.org/plasma/plasma-workspace/commit/97022f2f67c269cec2660ae4690e527cf452b322 |