Summary: | Double clicking icon on the desktop does not wait for a second click | ||
---|---|---|---|
Product: | [Plasma] plasmashell | Reporter: | Richard Tippl <richard.tippl> |
Component: | Folder | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | filip.kendes1, hein, nate |
Priority: | NOR | Keywords: | regression |
Version: | 6.1.5 | ||
Target Milestone: | 1.0 | ||
Platform: | Arch Linux | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/plasma/plasma-desktop/-/commit/a62e877c899c897b629783782046d4391e6c538d | Version Fixed In: | 6.2.0 |
Sentry Crash Report: |
Description
Richard Tippl
2024-09-14 15:04:14 UTC
Can confirm. Although important thing to note is that when you select the file, you need to double click on the file name to reproduce this, not the icon. Double clicking on the icon opens the file, but double clicking on the file name opens the "edit filename" mode. Double clicking on the file name in dolphin opens the file. A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2521 Git commit 36e0d2dd4198b7948adfb55d82c96852bba167de by Nate Graham, on behalf of Christoph Wolk. Committed on 16/09/2024 at 12:14. Pushed by ngraham into branch 'master'. containments/desktop: fix single-click to rename FolderView has a feature that allows the user to single-click the label of an already selected item to trigger renaming that item (given a particular combination of settings), mirroring similar functionality in Dolphin. This is, again as in Dolphin, supposed to not apply to cases where the label is double-clicked, which should open the double-clicked file. However, the detection of double-clicks does not work: it relies on the doubleClickInProgress property, which is set later in the event handler (and never reached in this case as the handler returns early), and so is almost always false (unless the clicked element changes mid- double-click). This change modifies the event handling logic to instead use the regular logic to wait for a double-click, and triggers the renaming function from the timer if applicable. FIXED-IN: 6.2 M +9 -7 containments/desktop/package/contents/ui/FolderView.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/36e0d2dd4198b7948adfb55d82c96852bba167de Git commit a62e877c899c897b629783782046d4391e6c538d by Nate Graham. Committed on 16/09/2024 at 12:23. Pushed by ngraham into branch 'Plasma/6.2'. containments/desktop: fix single-click to rename FolderView has a feature that allows the user to single-click the label of an already selected item to trigger renaming that item (given a particular combination of settings), mirroring similar functionality in Dolphin. This is, again as in Dolphin, supposed to not apply to cases where the label is double-clicked, which should open the double-clicked file. However, the detection of double-clicks does not work: it relies on the doubleClickInProgress property, which is set later in the event handler (and never reached in this case as the handler returns early), and so is almost always false (unless the clicked element changes mid- double-click). This change modifies the event handling logic to instead use the regular logic to wait for a double-click, and triggers the renaming function from the timer if applicable. FIXED-IN: 6.2 (cherry picked from commit 36e0d2dd4198b7948adfb55d82c96852bba167de) Co-authored-by: Christoph Wolk <christoph.wolk@gmx.de> M +9 -7 containments/desktop/package/contents/ui/FolderView.qml https://invent.kde.org/plasma/plasma-desktop/-/commit/a62e877c899c897b629783782046d4391e6c538d |