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

cant map any key to Alt-tab #80

Closed
dulm opened this issue May 5, 2020 · 2 comments · Fixed by #87
Closed

cant map any key to Alt-tab #80

dulm opened this issue May 5, 2020 · 2 comments · Fixed by #87

Comments

@dulm
Copy link

dulm commented May 5, 2020

version 0.2

add print to debug:

def send_key_action(key, action):
    print(key, action)
    update_modifier_key_pressed(key, action)
    update_pressed_keys(key, action)
    _uinput.write(ecodes.EV_KEY, key, action)
    send_sync()

linux mint 19.3 use Alt-tab to switch window.

map Shift-h to Alt-tab use K("Shift-h"): K("Alt-tab"),, not work.

Key.LEFT_SHIFT Action.PRESS
WM_CLASS 'jetbrains-pycharm' | active keymaps = [global, jetbrains-pycharm]
LShift-H
Key.LEFT_SHIFT Action.RELEASE

Key.LEFT_ALT Action.PRESS
Key.TAB Action.PRESS
Key.TAB Action.RELEASE
Key.LEFT_ALT Action.RELEASE

Key.LEFT_SHIFT Action.PRESS
Key.LEFT_SHIFT Action.RELEASE
@rbreaves
Copy link
Contributor

rbreaves commented May 7, 2020

This would be due to a commit I had submitted and merged into xkeysnail. I had not expected a keymap quite like this for app switching to be honest. I only expected Shift to be used as a temporary means to reverse a given Alt-Tab remap.

Seeing that was a bad assumption to make I will see about not ignoring Shift unless it is the singular modifier being mapped for the action.

Additionally the reason for the change was to improve the ability of mapping to Alt-Tab in general, the way it worked before wouldn't have allowed for the Alt-Tab menu to stay present because it was only quickly emitting the keymap versus keeping the modifier key held with the original. Unless the shift modifier worked differently than the others.

@rbreaves
Copy link
Contributor

rbreaves commented May 7, 2020

This issue is now fixed in the current pull request #76 with commit 113cc1a

I made other fixes for the patch and due to them this ignoring of the shift modifier is not needed.

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.

2 participants