Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

Fix/tab no modifiers #709

Merged
merged 2 commits into from
Aug 1, 2021
Merged

Fix/tab no modifiers #709

merged 2 commits into from
Aug 1, 2021

Conversation

tarkah
Copy link
Member

@tarkah tarkah commented Jul 31, 2021

Resolves #

Proposed Changes

  • Alt-tabbing shouldn't cause Ajour to change tabs

Checklist

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux
  • Added an entry to CHANGELOG.md if knowledge of this change could be valuable to users

@casperstorm casperstorm self-requested a review August 1, 2021 14:09
Copy link
Member

@casperstorm casperstorm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Also this tested this branch on macOS.

@tarkah tarkah changed the base branch from master to development August 1, 2021 15:11
@tarkah tarkah merged commit ef7bac0 into development Aug 1, 2021
@tarkah tarkah deleted the fix/tab-no-modifiers branch August 1, 2021 15:16
@Neurotoxin001
Copy link

Neurotoxin001 commented Aug 7, 2021

It's still not fixed even in 1.3.0
https://github.com/ajour/ajour/releases/tag/1.3.0

Windows 10 LTSB 1809

@tarkah
Copy link
Member Author

tarkah commented Aug 7, 2021

It's still not fixed even in 1.3.0

It appears the issue is when alt-tabbing back to Ajour. When alt-tabbing away from Ajour, the tab doesn't change. It seems the issue is when you alt-tab back to Ajour, and release the Tab key once Ajour is back in focus. This is the behavior on Windows, but the problem doesn't occur on Linux.

The bigger issue is we are capturing these hotkeys on key released, instead of key pressed. I believe switching over to key pressed should resolve this issue. @casperstorm any issues with making this change?

@tarkah
Copy link
Member Author

tarkah commented Aug 8, 2021

Hmm, changing to on key pressed actually makes this problem now occur on Linux as well.

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

Successfully merging this pull request may close these issues.

3 participants