Bug 356706

Summary: The system activity manager insists on opening partially off-screen
Product: [Plasma] plasmashell Reporter: Raman Gupta <rocketraman>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: bhush94, jazzvoid, plasma-bugs
Priority: NOR    
Version: 5.8.4   
Target Milestone: 1.0   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Screenshot
Output of xprop

Description Raman Gupta 2015-12-14 23:07:09 UTC
The system activity manager opens partially off-screen. I have a multi-monitor setup with 3 screens. The two left and right screens are smaller and centered off the middle screen. The default screen is the middle screen. The system activity manager opens partially visible on the left screen. After it is moved and closed, it does not remember the previous position (and kwin rules to that affect do not work).

Reproducible: Always

Steps to Reproduce:
1. Open System Activity
2.
3.

Actual Results:  
Displays partially off-screen

Expected Results:  
Should display completely on-screen

This installation is upgraded from KDE4.
Comment 1 Valerii Malov 2016-11-24 09:01:51 UTC
Gentoo, Plasma 5.8.3

I have a dual screen setup, systemmonitor always opens up on a secondary screen, seems to be related bug?

In my case offending line is geometry setting in KSystemActivityDialog::slotInit that disregards that geometry can have x1,y1 > 0 for multi screen setups
Comment 2 Raman Gupta 2016-11-24 15:01:56 UTC
Still happens in 5.7.5, QT 5.6.1.
Comment 3 Raman Gupta 2016-12-01 13:54:42 UTC
Still happening on Plasma 5.8.4, QT 5.6.2.

(In reply to Valeriy Malov from comment #1)
> I have a dual screen setup, systemmonitor always opens up on a secondary
> screen, seems to be related bug?

I think it is the same issue. If the screen opened on my main center monitor, it would be the correct size and position within that monitor. However, it opens on the wrong monitor.

It seems as if KDE is using the correct monitor to calculate the window's size and x,y position (relative to that monitor), but then actually placing the window on the wrong monitor.
Comment 4 David Edmundson 2016-12-11 23:33:54 UTC
Screenshots please.

Please also run "xprop" from a terminal and click on the window.

Then reset the bug status to unconfirmed.
Comment 5 Raman Gupta 2016-12-12 02:57:13 UTC
Created attachment 102735 [details]
Screenshot

Screenshot of window just after it opens via ksnapshot. Note its location on the left monitor, which is smaller than the center and right monitors.
Comment 6 Raman Gupta 2016-12-12 02:58:04 UTC
Created attachment 102736 [details]
Output of xprop

Output of xprop attached.
Comment 7 Valerii Malov 2018-03-11 14:36:14 UTC
Git commit ad5b9e98b1f255a83d04dfb5dbc8b31d32e91d53 by Valeriy Malov.
Committed on 11/03/2018 at 14:36.
Pushed by valeriymalov into branch 'master'.

Center systemmonitor window properly on multi-screen setup

Summary:
Do not force geometry or window hints when opening systemmonitor
This should let it position properly on a multi-monitor setup and reduce interference with user-set KWin preferences
Related: bug 368158

Reviewers: #plasma_workspaces, aacid, mart

Reviewed By: mart

Subscribers: davidedmundson, sebas, aacid, graesslin, plasma-devel

Tags: #plasma

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

M  +8    -42   systemmonitor/ksystemactivitydialog.cpp
M  +2    -3    systemmonitor/ksystemactivitydialog.h

https://commits.kde.org/plasma-workspace/ad5b9e98b1f255a83d04dfb5dbc8b31d32e91d53