Replace KScreen by QScreen for current window grab

Review Request #129127 - Created Oct. 8, 2016 and submitted

Information
Peter Wu
spectacle
master
e20026f...
Reviewers
kdegraphics
bgupta

libkscreen is overkill for querying purposes, rely on QScreen to find
the current screen under cursor.

Tested in a dualhead setting having screens of the following geometries:

QScreen(0x15a1040, name="eDP-1") QRect(2560,360 1920x1080)
QScreen(0x15a0e50, name="DP-2") QRect(0,0 2560x1440)

Option "Current Screen" does indeed follow the mouse cursor.

Tested with QT_SCALE_FACTOR=2, the cursor position seems right.

Note: test was done with Qt 5.7.0, but the QScreen APIs are made for Qt5. Also note that better high DPI support requires Qt 5.6 (Qt 5.6 also fixed the multi-monitor issues that plagued earlier Qt versions).

Issues

  • 0
  • 1
  • 0
  • 1
Description From Last Updated
Peter Wu
Luigi Toscano
Boudhayan Gupta
Peter Wu
Peter Wu
Sebastian Kügler
Boudhayan Gupta
Peter Wu
Peter Wu
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit e4c2e564a5b91497132d9a20d8f521af405286bd by Peter Wu to branch master.
Sebastian Kügler

Thanks, guys!

Loading...