Preserve relative position when moving to another display#1736
Closed
benoittgt wants to merge 1 commit intorxhanson:mainfrom
Closed
Preserve relative position when moving to another display#1736benoittgt wants to merge 1 commit intorxhanson:mainfrom
benoittgt wants to merge 1 commit intorxhanson:mainfrom
Conversation
Previously, moving a window to the next/previous display (or a specific display) would always center it on the destination screen when no prior Rectangle action was recorded (e.g. manually positioned windows). Fix: compute the window's proportional position on the source screen and apply it to the destination screen, clamping to stay within bounds. The `attemptMatchOnNextPrevDisplay` path and maximize special-case are unchanged. Fixes rxhanson#1723
Owner
|
Thanks for contributing! Finally got a chance to test this out. I'm thinking that this is intended, but going from right half on my laptop display to my next display resulted in: This result exposes ambiguity about my expectation of where the window would be anchored, and is unintuitive enough to me that my preference is to keep the existing functionality as the default behavior and have your change be something configured via Terminal command, similar to |
Author
|
I think the issue is somewhere else and with a conflict with Tahoe, as I can move window without Rectangle running. So I there there is a conflict. Closing. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Previously, moving a window to the next/previous display (or a specific display) would always center it on the destination screen when no prior Rectangle action was recorded (e.g. manually positioned windows).
Fix: compute the window's proportional position on the source screen and apply it to the destination screen, clamping to stay within bounds. The
attemptMatchOnNextPrevDisplaypath and maximize special-case are unchanged.Fixes #1723