Fixed window jumping - #8
Open
Wintaru wants to merge 2 commits into
Open
Conversation
added 2 commits
April 19, 2018 07:44
I know this is considered a bug in SublimeText, but the fix seemed simple enough. Thanks to fico for the code, which I found here: https://forum.sublimetext.com/t/change-language-lost-focus-annoying-behavior/18986/3
If you have the same file open in multiple groups, the previous fix wasn't enough, I think this is more robust now.
Author
|
To further clarify, the changes in 74502bc fixed an issue where focus would be changed to the wrong view open and also sometimes caused position changes in the wrong view. |
|
If this works, I would love for this to be merged. The jumping is really annoying. |
bubenkoff
requested changes
May 14, 2018
| group = window.active_group() | ||
| x, y = a_view.viewport_position() | ||
|
|
||
| print('group: {0}'.format(str(group))) |
| a_view.run_command('unexpand_tabs') | ||
|
|
||
| # Restore the viewport position and selections. | ||
| window.focus_group(group) |
Owner
There was a problem hiding this comment.
use extract the logic with restoring the position and selections into a context manager
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
3 participants
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.
I know this is considered a bug in SublimeText, but the fix seemed
simple enough. Thanks to fico for the code, which I found here:
https://forum.sublimetext.com/t/change-language-lost-focus-annoying-behavior/18986/3
This change is