Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve xinerama once more? #673

Open
uli42 opened this issue Mar 8, 2018 · 1 comment
Open

improve xinerama once more? #673

uli42 opened this issue Mar 8, 2018 · 1 comment
Assignees

Comments

@uli42
Copy link
Member

uli42 commented Mar 8, 2018

I have found another problem with the xinerama code, see attached image.

The light blue area in the lower left screen of the screenshot is invisible in reality on my setup because the left screen is only 1080 in height. That's all ok. The problem is, that our (fixed in #670) xinerama code should increase the xinerama height of the left screen to the full 900 because is partly in the invisible area (I hope you understand the explanation).
xinerama_1080_and_1200

I am not sure about the correct solution. Sunweaver's suggestion about an inner bounding box (see #670) could probably help here but will it help in any case?

I think we need to check these cases after the intersection
a) intersection edge is within the inner bounding box - nothing to do, take intersection result
b) intersection edge is outside the inner bounding box change intersection - increase height

(I have just tried and failed; turns out calculation of the inner bounding box is more complicated than it appears on first thought.)

@uli42
Copy link
Member Author

uli42 commented Sep 11, 2018

Ionic reimplemented xinerama to solve just that (and similar) problem(s), see #682

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants