D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-14 Thread Victor Dodon
printesoi added a comment. @zzag yes, but as per the https://doc.qt.io/qt-5/qscreen.html#availableGeometry-prop, it does not work on setups with multiple displays :) REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D29569 To: printesoi, davidedmundson, #kwin,

D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-14 Thread Vlad Zahorodnii
zzag accepted this revision. zzag added a comment. This revision is now accepted and ready to land. In general, it's better to use QScreen::availableGeometry(). REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D29569 To: printesoi, davidedmundson, #kwin, zzag

D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-09 Thread Victor Dodon
printesoi added a comment. I would like to add some more details about my setup: I have a laptop with a 4K display and I use a second 4K external monitor. I have enabled the 200% scalling and `xrandr -q | grep -E '\bconnected' `gives DP-0 connected 3840x2160+0+0 (normal left inverted

D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-09 Thread Anthony Fieroni
anthonyfieroni added a reviewer: zzag. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D29569 To: printesoi, davidedmundson, #kwin, zzag Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-09 Thread Victor Dodon
printesoi added reviewers: davidedmundson, KWin. REPOSITORY R278 KWindowSystem REVISION DETAIL https://phabricator.kde.org/D29569 To: printesoi, davidedmundson, #kwin Cc: kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns

D29569: Fix computing display geometry on multi-monitor HiDPI setups on X11

2020-05-09 Thread Victor Dodon
printesoi created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. printesoi requested review of this revision. REVISION SUMMARY KWindowSystem::workArea() and KWindowSystem::workArea(QList()) should return the same value, but the second call