Nulo/qmk_firmware
1
0
Fork 0
Code Releases Activity

Merge pull request #239 from DidierLoiseau/issue-221

Browse source 
Fix issue #221: LGUI(KC_LSFT) does not work
This commit is contained in:
Erez Zukerman 2016-04-06 22:37:02 +03:00
commit 27ee47d0bd
1 changed files with 12 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
tmk_core/common/action.c
View file

@ -88,14 +88,24 @@ void process_action(keyrecord_t *record)
action.key.mods<<4; action.key.mods<<4;
if (event.pressed) { if (event.pressed) {
if (mods) { if (mods) {
add_weak_mods(mods); if (IS_MOD(action.key.code)) {
// e.g. LSFT(KC_LGUI): we don't want the LSFT to be weak as it would make it useless.
// this also makes LSFT(KC_LGUI) behave exactly the same as LGUI(KC_LSFT)
add_mods(mods);
} else {
add_weak_mods(mods);
}
send_keyboard_report(); send_keyboard_report();
} }
register_code(action.key.code); register_code(action.key.code);
} else { } else {
unregister_code(action.key.code); unregister_code(action.key.code);
if (mods) { if (mods) {
del_weak_mods(mods); if (IS_MOD(action.key.code)) {
del_mods(mods);
} else {
del_weak_mods(mods);
}
send_keyboard_report(); send_keyboard_report();
} }
} }