Bug 365246 - windows in Icon-only Task manager and Task manager won't show in the right screens
Summary: windows in Icon-only Task manager and Task manager won't show in the right sc...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: Task Manager and Icons-Only Task Manager (show other bugs)
Version: master
Platform: Arch Linux Linux
: NOR grave
Target Milestone: 1.0
Assignee: Eike Hein
URL: https://dl.dropboxusercontent.com/u/1...
Keywords:
: 356994 365360 367947 368840 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-07-08 17:37 UTC by xmakerenx
Modified: 2016-10-07 06:25 UTC (History)
16 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 xmakerenx 2016-07-08 17:37:18 UTC
I am using archlinux 4.6.3-1-ARCH with 3 screens connected to my computer and after I have updated to the latest version of plasma 5 (5.7) every time I open a window on my middle screen it the task manager of the left screen , and every time I open a window in my left screen it shows as I have opened it in the middle screen

also when I right click on items at the task manager the menu shows in the wrong screen.

Reproducible: Always

Steps to Reproduce:
1. open dolphin  on the middle screen.

Actual Results:  
dolphin icon shows in the task manager in the left screen

Expected Results:  
dolphin icon shows in the task manager in the middle screen where it is opened

This bug makes using plasma 5 with multiple screens completely broken,
also right clicking on items in the task manager shows in the wrong screen.
This bug seem to be in both task manager and Icon-only task manager.
Comment 1 walmartshopper 2016-07-08 17:59:15 UTC
Same here on Arch after upgrading to plasma 5.7.0.  I have 4 monitors with a panel on each one.  Each panel has its own task manager with the option "only show tasks from the current screen" and the tasks all show on the wrong panels.  For now I had to turn off "only show from the current screen" to make it usable, but that's not very usable either with 4 screens.
Comment 2 Jason Wu 2016-07-09 22:32:04 UTC
Experiencing the same issue on Arch following the plasma 5.7.0-2 update. 4 monitors are configured with icon-only task manager to "only show tasks from current screen". Only primary monitor functions properly with all the rest of the windows appearing on the wrong task manager. Problem is also reproducible with the non-icon-only task manager. Deleting and re-creating the default panel does not have any affect.
Comment 3 Jason Wu 2016-07-12 07:21:40 UTC
(In reply to Jason Wu from comment #2)
> Experiencing the same issue on Arch following the plasma 5.7.0-2 update. 4
> monitors are configured with icon-only task manager to "only show tasks from
> current screen". Only primary monitor functions properly with all the rest
> of the windows appearing on the wrong task manager. Problem is also
> reproducible with the non-icon-only task manager. Deleting and re-creating
> the default panel does not have any affect.

Edit: I have found that the task manager filters work as expected if I only use 3 monitors  (disconnected the rightmost screen to replace projector lamp) but issue resumes once monitor is re-connected.
Comment 4 f.lupa 2016-07-12 07:37:21 UTC
(In reply to Jason Wu from comment #3)
> (In reply to Jason Wu from comment #2)
> > Experiencing the same issue on Arch following the plasma 5.7.0-2 update. 4
> > monitors are configured with icon-only task manager to "only show tasks from
> > current screen". Only primary monitor functions properly with all the rest
> > of the windows appearing on the wrong task manager. Problem is also
> > reproducible with the non-icon-only task manager. Deleting and re-creating
> > the default panel does not have any affect.
> 
> Edit: I have found that the task manager filters work as expected if I only
> use 3 monitors  (disconnected the rightmost screen to replace projector
> lamp) but issue resumes once monitor is re-connected.

I am using 3 monitors as well. For me however the behavior is that the center monitor works fine while the icons on the left and right monitor are swapped in their positions.
Comment 5 Eike Hein 2016-07-12 07:50:48 UTC
*** Bug 365360 has been marked as a duplicate of this bug. ***
Comment 6 Eike Hein 2016-07-12 07:51:44 UTC
If any of the reporters who can reliably reproduce their problems are able to recompile KDE C++ code with patches and are willing to use IRC for a debug session, please let me know.
Comment 7 Eike Hein 2016-07-12 09:09:05 UTC
Git commit 332b96b3632d0fc4b3bbd21907127093f0988e45 by Eike Hein.
Committed on 12/07/2016 at 08:55.
Pushed by hein into branch 'Plasma/5.7'.

Do screen filtering by screen geometry rather than number.

Summary:
Avoids problems stemming from libtaskmanager and plasma-framework's
AppletInterface::screen numbering screens differently (QScreen order
vs. who knows what).

Reviewers: #plasma, graesslin, mart

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D2143

M  +1    -1    libtaskmanager/abstracttasksmodel.h
M  +1    -5    libtaskmanager/startuptasksmodel.cpp
M  +11   -16   libtaskmanager/taskfilterproxymodel.cpp
M  +13   -12   libtaskmanager/taskfilterproxymodel.h
M  +1    -1    libtaskmanager/taskgroupingproxymodel.cpp
M  +6    -6    libtaskmanager/tasksmodel.cpp
M  +12   -12   libtaskmanager/tasksmodel.h
M  +33   -0    libtaskmanager/tasktools.cpp
M  +9    -0    libtaskmanager/tasktools.h
M  +1    -1    libtaskmanager/waylandtasksmodel.cpp
M  +3    -34   libtaskmanager/xwindowtasksmodel.cpp

http://commits.kde.org/plasma-workspace/332b96b3632d0fc4b3bbd21907127093f0988e45
Comment 8 Eike Hein 2016-07-12 09:11:34 UTC
Git commit 18bd19433db9a568f514ae4b22d9084710ead32a by Eike Hein.
Committed on 12/07/2016 at 09:11.
Pushed by hein into branch 'Plasma/5.7'.

Hand screen geometry rather than screen number to TasksModel.

M  +1    -1    applets/taskmanager/package/contents/ui/main.qml

http://commits.kde.org/plasma-desktop/18bd19433db9a568f514ae4b22d9084710ead32a
Comment 9 Eike Hein 2016-07-12 09:12:46 UTC
I believe the above commits should address these problems -- I'll keep the ticket open for now until some test results come around.
Comment 10 Jason Wu 2016-07-12 14:20:07 UTC
I'll recompile this weekend and see what happens. Thanks!
Comment 11 Eike Hein 2016-07-12 15:31:44 UTC
Unfortunately I had to revert the changes for now, as the 5.7.1 tagging was upon us. They worked nicely for some testers, but badly for another tester, for reasons I currently don't yet know/understand, so the safe choice was to go back to the old code (which also works fine for several testers). I'll get more debug info from the tester tomorrow, and see about what can be done for 5.7.2 one week from now.
Comment 12 Jason Wu 2016-07-12 17:01:36 UTC
For those experiencing this issue: a temporary workaround (if your monitors are side by side) is to disable everything except your leftmost screen and enable your monitors from left to right. This will force the re-numbering of screens.
Comment 13 Eike Hein 2016-07-13 15:19:06 UTC
Anyone who can: Please test the hein/screenGeometryFilter branches of plasma-workspace and plasma-desktop and see if they work better.
Comment 14 Craig 2016-07-19 02:20:59 UTC
Hi guys, can confirm same issue on arch as well with 4 monitors, the 3rd is my primary, 3rd and 4th task icons show correctly however on the 1st and 2nd it's reversed (1st shows on 2nd toolbar and vice versa). I'm out today but can recompile and test as needed or hop on IRC and do the same probably tomorrow! Will report back
Comment 15 Eike Hein 2016-08-11 09:19:59 UTC
*** Bug 356994 has been marked as a duplicate of this bug. ***
Comment 16 Eike Hein 2016-08-11 09:22:00 UTC
The code in the hein/screenGeometryFilter branches was since merged to master.
Comment 17 xmakerenx 2016-08-11 09:30:38 UTC
so this fix should be 5.7.4 ?
Comment 18 Eike Hein 2016-08-11 09:40:47 UTC
No, currently it's in the branch that will be come 5.8. It hasn't had enough testing yet to backport it to the 5.7 branch. I'm relying on people willing to test there.
Comment 19 marco 2016-08-29 14:05:56 UTC
Have that issue too witch my arch/kde. Looking forward for 5.8
Comment 20 David Edmundson 2016-08-29 15:18:34 UTC
*** Bug 367947 has been marked as a duplicate of this bug. ***
Comment 21 Eike Hein 2016-09-07 12:01:30 UTC
Any Neon Unstable testers ready to provide results?
Comment 22 Stephen 2016-09-10 00:48:01 UTC
Also having this issue...

Arch Linux, KDE 5.7.4


http://i.imgur.com/1fq8Xrr.png
Comment 23 Eike Hein 2016-09-15 11:02:06 UTC
*** Bug 368840 has been marked as a duplicate of this bug. ***
Comment 24 Patrick Boutilier 2016-09-29 13:11:36 UTC
(In reply to Eike Hein from comment #21)
> Any Neon Unstable testers ready to provide results?

neon-devedition-gitstable-20160928-1047-amd64.iso

4 monitors. After running kscreen to setup orientation and position I added 3 empty panels to the screens without panels and added the Task Manager widget to each one. Configured to "Show only tasks from the current screen" and it worked perfectly. The only problem I had was that the Task Manger on the default panel would not show anything on its screen unless it was configured to show tasks from all screens. But when I deleted that Task Manager and re-added it it worked as expected as well.
Comment 25 fri.K 2016-09-30 07:01:21 UTC
(In reply to Jason Wu from comment #12)
> For those experiencing this issue: a temporary workaround (if your monitors
> are side by side) is to disable everything except your leftmost screen and
> enable your monitors from left to right. This will force the re-numbering of
> screens.

It's annoying but surprisingly it's working! Thanks :)
version:
plasma-workspace  5.7.5-0ubuntu1~ubuntu16.04~ppa1
Comment 26 f.lupa 2016-10-06 19:03:57 UTC
The update to plasma 5.8.0 fixed this issue for me (Arch).
Comment 27 xmakerenx 2016-10-07 06:18:09 UTC
I can also confirm that updating to plasma 5.8.0 has fixed the issue (I am also on Arch)
Comment 28 Eike Hein 2016-10-07 06:25:49 UTC
Great to hear! Closing now.