<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>493124</bug_id>
          
          <creation_ts>2024-09-14 15:04:14 +0000</creation_ts>
          <short_desc>Double clicking icon on the desktop does not wait for a second click</short_desc>
          <delta_ts>2024-09-17 15:32:11 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Plasma</classification>
          <product>plasmashell</product>
          <component>Desktop icons &amp; Folder View widget</component>
          <version>6.1.5</version>
          <rep_platform>Arch Linux</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>regression</keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>1.0</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Richard Tippl">richard</reporter>
          <assigned_to name="Plasma Bugs List">plasma-bugs-null</assigned_to>
          <cc>filip.kendes1</cc>
    
    <cc>hein</cc>
    
    <cc>nate</cc>
          
          <cf_commitlink>https://invent.kde.org/plasma/plasma-desktop/-/commit/a62e877c899c897b629783782046d4391e6c538d</cf_commitlink>
          <cf_versionfixedin>6.2.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>2353836</commentid>
    <comment_count>0</comment_count>
    <who name="Richard Tippl">richard</who>
    <bug_when>2024-09-14 15:04:14 +0000</bug_when>
    <thetext>SUMMARY
Double clicking a desktop icon that has been before selected tries to rename the file instead of opening.

STEPS TO REPRODUCE
1. Create a file on the desktop
2. Click the file to select it and wait a while
3. Any time after that and without deselecting, try to double click the icon to open

OBSERVED RESULT
The first click immediately enters &quot;edit filename&quot; mode and the file is not opened.

EXPECTED RESULT
Following the first click, there should be a delay and wait for a second click. Enter edit mode when second click is not detected.

SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 6.1.5
KDE Frameworks Version: 6.5.0
Qt Version: 6.7.2
Kernel Version: 6.10.9-arch1-2 (64-bit)
Graphics Platform: Wayland

ADDITIONAL INFORMATION
The &quot;expected result&quot; behavior is exactly how Dolphin handles this situation.
If you double click a selected file, the file opens, if you single click it, it enters name edit mode after a small moment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2353843</commentid>
    <comment_count>1</comment_count>
    <who name="Filip">filip.kendes1</who>
    <bug_when>2024-09-14 15:54:44 +0000</bug_when>
    <thetext>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 &quot;edit filename&quot; mode.

Double clicking on the file name in dolphin opens the file.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2354519</commentid>
    <comment_count>2</comment_count>
    <who name="Bug Janitor Service">bug-janitor</who>
    <bug_when>2024-09-15 10:21:10 +0000</bug_when>
    <thetext>A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2521</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2354758</commentid>
    <comment_count>3</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-09-16 12:22:42 +0000</bug_when>
    <thetext>Git commit 36e0d2dd4198b7948adfb55d82c96852bba167de by Nate Graham, on behalf of Christoph Wolk.
Committed on 16/09/2024 at 12:14.
Pushed by ngraham into branch &apos;master&apos;.

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</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>2354768</commentid>
    <comment_count>4</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2024-09-16 12:49:20 +0000</bug_when>
    <thetext>Git commit a62e877c899c897b629783782046d4391e6c538d by Nate Graham.
Committed on 16/09/2024 at 12:23.
Pushed by ngraham into branch &apos;Plasma/6.2&apos;.

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 &lt;christoph.wolk@gmx.de&gt;

M  +9    -7    containments/desktop/package/contents/ui/FolderView.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/a62e877c899c897b629783782046d4391e6c538d</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>