Bug 360219 - Right Click opens files
Summary: Right Click opens files
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Folder (show other bugs)
Version: 5.5.5
Platform: openSUSE Linux
: NOR normal
Target Milestone: 1.0
Assignee: Eike Hein
URL:
Keywords:
: 368793 375853 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-03-07 17:28 UTC by Florian Röhrer
Modified: 2017-02-01 17:28 UTC (History)
12 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.8.6
Sentry Crash Report:


Attachments
behaviour of right clicking (1.69 MB, video/mkv)
2016-04-06 21:33 UTC, Florian Röhrer
Details
Possible patch (563 bytes, patch)
2016-12-28 12:57 UTC, Kai Uwe Broulik
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Röhrer 2016-03-07 17:28:46 UTC
Often if i do a right click on a file it opens the file and simultaneously the shortcut menu appears. 

Reproducible: Sometimes
Comment 1 Florian Röhrer 2016-04-01 18:04:41 UTC
Is this bug going to be fixed in the near future? It is really annoying and happens nearly every time. 
It only appears for files which are on the Desktop (folder view)
Comment 2 Eike Hein 2016-04-02 15:51:07 UTC
Folder bugs can't be fixed if they're reported against the wrong component and the reports aren't seen.
Comment 3 Eike Hein 2016-04-02 15:51:50 UTC
Do you use single click or double click?
Comment 4 Florian Röhrer 2016-04-02 15:54:26 UTC
I use single click. For the future...where should i have reported this bug?
Comment 5 Eike Hein 2016-04-02 15:55:16 UTC
Against the "Folder" component of product "plasmashell".
Comment 6 Florian Röhrer 2016-04-03 18:24:49 UTC
Do you need any further information because the status of this bug is "NEEDSINFO WAITINGFORINFO"?
Comment 7 Eike Hein 2016-04-03 18:25:58 UTC
Not at the moment, thanks.
Comment 8 Eike Hein 2016-04-05 12:24:50 UTC
I can't reproduce this. The code changed a bit since 5.5.5; please reopen if the problem persists with 5.6.x.
Comment 9 Florian Röhrer 2016-04-05 13:54:44 UTC
At the moment I have Plasma 5.6.1 installed and it still happens...
Comment 10 Eike Hein 2016-04-06 08:23:57 UTC
Could you record and attach a short video using e.g. vokoscreen? It might give me a better clue about what's going on.
Comment 11 Florian Röhrer 2016-04-06 21:33:27 UTC
Created attachment 98274 [details]
behaviour of right clicking

you can see the behaviour on second 5 of the video...
Comment 12 Florian Röhrer 2016-04-06 21:33:53 UTC
I only did a single right click on the file
Comment 13 Eike Hein 2016-04-07 08:39:24 UTC
Thank you, that's helpful. I'll try to get around to this problem soon.
Comment 14 Chris Williamson 2016-05-04 18:48:03 UTC
This bug just started happening to me after I upgraded Kubuntu to version 16.04 (from 15.10).
Comment 15 Wolfgang Bauer 2016-07-21 14:07:29 UTC
This still happens in 5.7.2.

Note that the file is only opened if you right-click and immediately release the mouse button.
If you keep it pressed, the file is never opened, only the context menu appears.
So that's at least a workaround.

Maybe related: if you drag and drop an entry from the application menu to the folderview it is also "opened". You can see the popup that asks whether you want to copy/move/symlink for a second but it disappears again before you can actually choose anything and the application is started instead.
Only seems to happen with Kicker though, not Kickoff.
Comment 16 Eike Hein 2016-07-23 00:41:03 UTC
The latter is actually a bug in Kicker it seems.
Comment 17 Eike Hein 2016-07-23 00:46:32 UTC
Git commit 0065d3548c7db695b06d80250b11d018214967dc by Eike Hein.
Committed on 23/07/2016 at 00:46.
Pushed by hein into branch 'Plasma/5.7'.

Don't activate it on release events when a drag was started prior to it.

M  +1    -0    applets/kicker/package/contents/ui/ItemListDelegate.qml
M  +1    -0    applets/kicker/package/contents/ui/SideBarItem.qml

http://commits.kde.org/plasma-desktop/0065d3548c7db695b06d80250b11d018214967dc
Comment 18 kdebugs 2016-07-25 01:31:04 UTC
Possibly related:  Today on Manjaro w/ plasma 5.6.5 I had desktop icons in folder view layout launching with just a single right-click.  No menu displayed - just instantly launched the application.  I realize this is somewhat different from the bug reported above, but it's similar and I couldn't reproduce it after logging out and back in, so I won't file a separate report.
Comment 19 Wolfgang Bauer 2016-07-25 11:55:36 UTC
(In reply to Eike Hein from comment #16)
> The latter is actually a bug in Kicker it seems.

Ok, sorry.
I can confirm that your commit fixes that problem.
Thanks.
Comment 20 Eike Hein 2016-07-25 11:59:59 UTC
No need to be sorry, thanks for the report!

I still want to address these right-click woes, but it's being hampered by the fact that I personally can't reproduce it and the code does the right thing already (the click handler checks for the right mouse button and balks out). This might need more complicated state to reproduce (a la bug 366065) or be a Qt bug (lying about which button was used in the event handler).
Comment 21 Wolfgang Bauer 2016-07-25 12:12:11 UTC
(In reply to kdebugs from comment #18)
> Possibly related:  Today on Manjaro w/ plasma 5.6.5 I had desktop icons in
> folder view layout launching with just a single right-click.  No menu
> displayed - just instantly launched the application.  I realize this is
> somewhat different from the bug reported above, but it's similar and I
> couldn't reproduce it after logging out and back in, so I won't file a
> separate report.

It may have been the same problem, though in my experience the context menu does show up, only the file is opened at the same time.

Maybe your right-handed/left-handed mouse settings got temporarily messed up for some reason? ;-)
Comment 22 Wolfgang Bauer 2016-07-25 12:16:19 UTC
(In reply to Eike Hein from comment #20)
> No need to be sorry, thanks for the report!

Ok, but normally I would have opened a separate bug report. ;-)
It just seemed to have the same cause...

> I still want to address these right-click woes, but it's being hampered by
> the fact that I personally can't reproduce it and the code does the right
> thing already (the click handler checks for the right mouse button and balks
> out). This might need more complicated state to reproduce (a la bug 366065)
> or be a Qt bug (lying about which button was used in the event handler).

It's quite reliably reproducible here (since 5.5 or so).
I plan to investigate further myself, just didn't have time yet.

For the record, this happened with Qt 5.6 and still happens with 5.7.
I'm not sure if it was a problem with 5.5 too, but I will check.
Comment 23 kdebugs 2016-07-25 16:46:00 UTC
(In reply to Wolfgang Bauer from comment #21)
> It may have been the same problem, though in my experience the context menu
> does show up, only the file is opened at the same time.

In the upload provided in comment #11, you can see that the context menu opens but immediately disappears.  The appearance or lack of the menu may just come down to the speed and current load of the particular machine.  I can quite reliably reproduce this bug by continually left-clicking the desktop followed by right-clicking the icon repeatedly until it launches.  When it does launch, a context menu does NOT appear for me, however when I tried to capture that with my screen recorder, it slowed things down enough that the context menu did momentarily appear, as in the attachment already uploaded.

I did have it happen on the first right-click once today - what was most different yesterday was that it was happening first time, every time, until I logged out.

> Maybe your right-handed/left-handed mouse settings got temporarily messed up
> for some reason? ;-)

Nothing else was backward, and I would have noticed if I got a context menu with a left-click.
Comment 24 Wolfgang Bauer 2016-07-25 18:26:59 UTC
(In reply to kdebugs from comment #23)
> (In reply to Wolfgang Bauer from comment #21)
> > Maybe your right-handed/left-handed mouse settings got temporarily messed up
> > for some reason? ;-)
> 
> Nothing else was backward, and I would have noticed if I got a context menu
> with a left-click.

All right, that was just a though.
And just to be clear, I didn't want to deny the existance of this bug, rather the opposite. As I wrote, I can reproduce it quite reliable here, on two different systems and in a VM. But in my case, the context menu does reliably appear and stays.

(In reply to Wolfgang Bauer from comment #22)
> For the record, this happened with Qt 5.6 and still happens with 5.7.
> I'm not sure if it was a problem with 5.5 too, but I will check.

I can reproduce it in a VM with openSUSE Leap 42.1 too, which uses Plasma 5.5.5 and Qt 5.5.1.
So it apparently is not caused by a change or bug in Qt 5.6 at least...
Comment 25 Eike Hein 2016-07-29 14:05:11 UTC
This is actually caused by a Qt bug; my fix for it is stuck since a while ago - https://codereview.qt-project.org/#/c/146786/
Comment 26 kdebugs 2016-07-29 17:56:50 UTC
(In reply to Eike Hein from comment #25)
> This is actually caused by a Qt bug; my fix for it is stuck since a while
> ago - https://codereview.qt-project.org/#/c/146786/

In reading that bug, the "common example" given at the top describes "clicking outside the popup, and then clicking in the same spot that was clicked to dismiss the popup."  That sounds a whole lot more like Bug 366065 than this bug.  To be clear, this bug describes files being opened exactly on the right-click of that file icon, and NOT on clicking elsewhere to dismiss the popup menu.

Likewise, the other example given is always reproducible: "on the desktop surface: Right-click empty space to get context menu, right-click above [i.e. on] desktop icon to dismiss menu, right-click icon again and you still get the empty-space context instead of the icon one."
That explains how you might get the wrong context menu in right-clicking a desktop icon, but how does it explain opening that file/link when you right-click it?

Also, that Qt bug and Bug 366065 are always reproducible, whereas this bug is only "Reproducible: Sometimes", which makes this bug seem more like a timing problem, race condition, or something dependent on less obvious circumstances.
Comment 27 Eike Hein 2016-07-30 15:48:58 UTC
My assumption is that this bug only happens after the state previously got corrupted by using the context menu elsewhere. This is not included in any of the steps-to-reproduce, but I can't reproduce the problem otherwise, and reading the code I can't see how it would happen otherwise - unless there's a different Qt bug that causes mouse.buttons in MouseArea's clicked() handler to be wrong, which is also possible.
Comment 28 Eike Hein 2016-07-31 14:34:59 UTC
Here's an updated patch for Qt 5.7 if you want to test: https://codereview.qt-project.org/#/c/166455/
Comment 29 Wolfgang Bauer 2016-08-01 15:32:35 UTC
(In reply to Eike Hein from comment #27)
> My assumption is that this bug only happens after the state previously got
> corrupted by using the context menu elsewhere.

Well, in my case this occurs right after login, without doing anything else first.
I.e. I just right-click on an icon in the folderview after logging in and it gets opened (in addition to showing the context menu).

(In reply to Eike Hein from comment #28)
> Here's an updated patch for Qt 5.7 if you want to test:
> https://codereview.qt-project.org/#/c/166455/

I tried it and it does NOT help with this problem.

Btw, Qt 5.7.0 failed to compile with this patch here, I had to remove the "0L" parameter in the call to deliverHoverEvent().
Comment 30 Wolfgang Bauer 2016-08-13 15:01:52 UTC
I noticed meanwhile that this only happens if I tap the right mouse button extremely shortly.
If I keep it pressed for a "longer" period (maybe 0.1 seconds, hard to say exactly...), the file/icon is *not* opened.

Maybe this helps in reproducing the issue...
Comment 31 Tsu Jan 2016-09-11 20:04:11 UTC
For me, both on a desktop computer with nvidia and on a laptop with Intel, the only way of right clicking an item on the desktop without opening it is holding the right mouse button for 1-2 seconds (Manjaro Testing with plasma-workspace-5.7.4).
Comment 32 P.Otato 2016-09-20 07:09:40 UTC
*** Bug 368793 has been marked as a duplicate of this bug. ***
Comment 33 P.Otato 2016-09-20 07:22:24 UTC
Hey,

I filed a report on the same topic (Bug 368793) and found this bug report later. Therefore I marked my bug report as duplicate of this one.
The problem exists for me in Plasma 5.5.5 (openSUSE Leap 42.1) as well as in Plasma 5.8 (KDE Neon GIT-stable branch) and can be always be reproduced. It happens when the right mouse button is clicked very fast, the icon is not selected previously (although I'm not sure if this makes a difference). The problem does not exist with slow right mouse button clicks.

I used xev to make sure that no mouse movement is involved or others buttons are pressed simultaneously. It is reproducible with different usb mice. The bug was also tested on a notebook using the touchpad keys to be certain there is no pointer movement.
Comment 34 Michael D 2016-10-16 19:37:43 UTC
The bug is present in plasm a 5.8.1 (Neon Dev stable). I have a folder view widget in my panel. If i invoke it and right click an item, the item opens and the right click menu appears at the same time.
Comment 35 Aurélien Murith 2016-11-20 17:26:29 UTC
Problem still present on Plasma 5.8.2 (Qt 5.7).

Right-clicking on files that are on the desktop (folder view) often (not always) opens them, in addition of opening the context menu.

It's a really annoying bug...
Comment 36 Kai Uwe Broulik 2016-12-28 12:57:06 UTC
Created attachment 103044 [details]
Possible patch

Can you please test the following patch?

I can sometimes trigger the bug and turns out that mouse.buttons is 0 for clicked events whereas mouse.button is properly set. It should bail out when released with right mouse button as that opens the context menu but the condition is never met.

According to documentation this looks intentional, at least it says "For mouse move events […]. For mouse press and double click events […]. For mouse release events […]." I can see no mention of regular "clicked", no idea why that is, though.
Comment 37 Wolfgang Bauer 2016-12-28 18:37:09 UTC
(In reply to Kai Uwe Broulik from comment #36)
> Created attachment 103044 [details]
> Possible patch
> 
> Can you please test the following patch?

I tried it, and the patch does indeed fix the problem here.
Comment 38 Eike Hein 2017-01-03 07:50:54 UTC
Patch seems fine, please commit Kai.
Comment 39 Kai Uwe Broulik 2017-01-09 07:30:18 UTC
Git commit d2fde361d3c8fb40fb6c1e53e4178042799b6691 by Kai Uwe Broulik.
Committed on 09/01/2017 at 07:28.
Pushed by broulik into branch 'Plasma/5.8'.

[Folder View] Fix right click erroneously opening files

It turns out mouse.buttons is 0 for "clicked" events whereas mouse.button is properly set.
It should bail out when released with right mouse button as that opens the context menu
but this condition is never met.

According to documentation this looks intentional, at least it says "For mouse move
events […]. For mouse press and double click events […]. For mouse release events […]."
I can see no mention of regular "clicked", no idea why that is, though.
FIXED-IN: 5.8.6

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

https://commits.kde.org/plasma-desktop/d2fde361d3c8fb40fb6c1e53e4178042799b6691
Comment 40 Kai Uwe Broulik 2017-02-01 17:28:15 UTC
*** Bug 375853 has been marked as a duplicate of this bug. ***