https://bugs.kde.org/show_bug.cgi?id=386099

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
      Latest Commit|                            |https://commits.kde.org/kwi
                   |                            |n/5d7e22e5dcf8981f9ba65088b
                   |                            |81c1c6d456efef3
         Resolution|---                         |FIXED

--- Comment #13 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 5d7e22e5dcf8981f9ba65088b81c1c6d456efef3 by David Edmundson.
Committed on 25/10/2017 at 22:13.
Pushed by davidedmundson into branch 'Plasma/5.11'.

Fix viewport on vertically stacked monitors

Summary:
We want to translate by the monitor position, so that needs to be
the negative of the position.

But Kwin/KScreen treats 0 as the top of all monitors. GL treats 0 as
bottom, so that all needs inverting.

Hence this should be a positive y value for the viewport.
Related: bug 385655

Test Plan:
Had two monitors
Side by side was - fine
Stacked vertically - still fine

Modded X code to extend in y instead of x.
3 monitors worked fine.

Nested wayland only seems to support one screen?

Reviewers: #plasma

Subscribers: plasma-devel, kwin, #kwin

Tags: #kwin

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

M  +1    -1    plugins/platforms/drm/egl_gbm_backend.cpp
M  +1    -1    plugins/platforms/x11/windowed/egl_x11_backend.cpp

https://commits.kde.org/kwin/5d7e22e5dcf8981f9ba65088b81c1c6d456efef3

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to