Bug 320758 - Clicking on systray bluetooth icon on second monitor produces a pop-up menu on first monitor
Summary: Clicking on systray bluetooth icon on second monitor produces a pop-up menu o...
Status: RESOLVED FIXED
Alias: None
Product: solid
Classification: Frameworks and Libraries
Component: bluetooth-systray (show other bugs)
Version: 4.10.3
Platform: Gentoo Packages Linux
: NOR normal
Target Milestone: ---
Assignee: Alex Fiestas
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-06-05 10:01 UTC by Dave Flogeras
Modified: 2014-05-13 11:41 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dave Flogeras 2013-06-05 10:01:46 UTC
As the summary says, the popup dialog always opens on my first (primary) display when I click on the icon on the second display.

Using bluedevil-1.3.1 with libbluedevil-1.9.3 against KDE 4.10.3

This was also and issue with bluedevil-1.3, libbluedevil-1.9.2 and KDE 4.10.2/1

Reproducible: Always
Comment 1 Alex Fiestas 2013-08-01 20:01:38 UTC
This is not our fault, but Qt's or KWin but certanly not us (we don't control where the popup appears).

Is this still always reproducible?

Aaron, Notmart, Martin any reason why this could happen on your side?
Comment 2 Marco Martin 2013-08-01 20:09:36 UTC
is it a statusnotifier systray icon?

i see here that it should use Applet::PopupPosition() to compute the position, that should work...
Comment 3 Dave Flogeras 2013-08-01 21:18:10 UTC
Yup, still happens with KDE 4.10.5 and bluedevil-1.3.1 and libbluedevil-1.9.3
Comment 4 Alex Fiestas 2014-01-19 20:43:51 UTC
Do you have 2 systemtrays one on each screen?
Comment 5 Marco Martin 2014-01-19 21:25:07 UTC
Git commit 15ad062839d32613c3bd3ca96acf40b5f59e8a44 by Marco Martin.
Committed on 19/01/2014 at 21:22.
Pushed by mart into branch 'master'.

dbusmenu is async: systrays will race to show menu

don't show a menu if __processClick happened more than one second ago.
this makes the systray the did not generate the click to ignore the menu ready signal.
It's a pretty bad solution, it would need something better.

M  +8    -24   plasma/desktop/qmlpackages/desktop/contents/applet/CompactApplet.qml

http://commits.kde.org/kde-workspace/15ad062839d32613c3bd3ca96acf40b5f59e8a44
Comment 6 Marco Martin 2014-01-19 21:28:38 UTC
Git commit 77b0acde04aeffcca2b713b6d931230840d517c0 by Marco Martin.
Committed on 19/01/2014 at 21:26.
Pushed by mart into branch 'KDE/4.11'.

dbusmenu is async: systrays will race to show menu

    don't show a menu if __processClick happened more than one second ago.
    this makes the systray the did not generate the click to ignore the menu ready signal.
    It's a pretty bad solution, it would need something better.

M  +10   -1    plasma/generic/applets/systemtray/package/contents/ui/StatusNotifierItem.qml

http://commits.kde.org/kde-workspace/77b0acde04aeffcca2b713b6d931230840d517c0
Comment 7 Alex Fiestas 2014-03-09 01:45:51 UTC
Tested, can't reproduce so it is fixed :)
Comment 8 Dave Flogeras 2014-03-09 17:03:42 UTC
Still broken here, using bluez 5.15 and (lib)bluedevil-2.0_rc1 on Gentoo.  Can reproduce without a second monitor, just add a second "default panel" to your workspace (say at the top, so that both are visiable).  Click on the bluetooth icon in both systrays.
Comment 9 Alex Fiestas 2014-03-10 15:14:45 UTC
KDE version?
(In reply to comment #8)
> Still broken here, using bluez 5.15 and (lib)bluedevil-2.0_rc1 on Gentoo. 
> Can reproduce without a second monitor, just add a second "default panel" to
> your workspace (say at the top, so that both are visiable).  Click on the
> bluetooth icon in both systrays.
Comment 10 Dave Flogeras 2014-03-10 15:26:04 UTC
Currently 4.11.5
Comment 11 Christoph Feck 2014-04-21 11:23:11 UTC
To get the fix from comment #6, you would need KDE Workspaces 4.11.6. If this still does not work correctly with that version, please add a comment.
Comment 12 Dave Flogeras 2014-05-13 11:41:13 UTC
Works as expected after recent upgrade to 4.12.5, thank you!