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

Bug in multi-monitor positioning code #5

Open
CaiusJard opened this issue Sep 7, 2022 · 0 comments · May be fixed by #6
Open

Bug in multi-monitor positioning code #5

CaiusJard opened this issue Sep 7, 2022 · 0 comments · May be fixed by #6

Comments

@CaiusJard
Copy link

CaiusJard commented Sep 7, 2022

If one has a screen arrangement such as:

image

with e.g. monitor 3 being the primary, it means the fullScreen.Left bound is negative

image

The calc for the horizontal center thus needs to include the left bound as a starting point:

   var centerOfAllScreensX = fullScreen.Left + fullScreen.Width/2.0;

Line 25 of Dragger.cs just has var centerOfAllScreensX = fullScreen.Width/2.0; which I believe only works as intended if the fullscreen bound starts from a Left of 0.

@DanielRose DanielRose linked a pull request Sep 29, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant