aboutsummaryrefslogtreecommitdiff
path: root/quantum/process_keycode
diff options
context:
space:
mode:
authorDrashna Jaelre <drashna@live.com>2021-01-04 16:37:20 -0800
committerGitHub <noreply@github.com>2021-01-04 16:37:20 -0800
commit810eafad121bda333c53490e2d8a29f3a83d9c19 (patch)
tree975b8bd9c846d13f30da944e873326e0e0f69fa8 /quantum/process_keycode
parentc38fe492426676cf101eeb024f7f33d8e98c445f (diff)
downloadqmk_firmware-810eafad121bda333c53490e2d8a29f3a83d9c19.tar.gz
qmk_firmware-810eafad121bda333c53490e2d8a29f3a83d9c19.zip
Fix Tap-Hold Configs (#11127)
* Add proper prototypes for Tap-Hold Per Key functions * Fix handwired/tennie default keymap * Remove unneeded references * Fix tapping term per key check in space cadet * Pre-emptive fix for tap dance * Fix marksard/leftover30 * Replace hard coded tapping term with define
Diffstat (limited to 'quantum/process_keycode')
-rw-r--r--quantum/process_keycode/process_space_cadet.c11
-rw-r--r--quantum/process_keycode/process_space_cadet.h3
-rw-r--r--quantum/process_keycode/process_tap_dance.c5
3 files changed, 10 insertions, 9 deletions
diff --git a/quantum/process_keycode/process_space_cadet.c b/quantum/process_keycode/process_space_cadet.c
index bcaf62a96..f99db2a87 100644
--- a/quantum/process_keycode/process_space_cadet.c
+++ b/quantum/process_keycode/process_space_cadet.c
@@ -16,10 +16,6 @@
16#include "process_space_cadet.h" 16#include "process_space_cadet.h"
17#include "action_tapping.h" 17#include "action_tapping.h"
18 18
19#ifdef NO_ACTION_TAPPING
20__attribute__((weak)) uint16_t get_tapping_term(uint16_t keycode, keyrecord_t *record) { return TAPPING_TERM; };
21#endif
22
23// ********** OBSOLETE DEFINES, STOP USING! (pls?) ********** 19// ********** OBSOLETE DEFINES, STOP USING! (pls?) **********
24// Shift / paren setup 20// Shift / paren setup
25#ifndef LSPO_KEY 21#ifndef LSPO_KEY
@@ -97,7 +93,12 @@ void perform_space_cadet(keyrecord_t *record, uint16_t sc_keycode, uint8_t holdM
97 register_mods(MOD_BIT(holdMod)); 93 register_mods(MOD_BIT(holdMod));
98 } 94 }
99 } else { 95 } else {
100 if (sc_last == holdMod && timer_elapsed(sc_timer) < get_tapping_term(sc_keycode, record)) { 96#ifdef TAPPING_TERM_PER_KEY
97 if (sc_last == holdMod && timer_elapsed(sc_timer) < get_tapping_term(sc_keycode, record))
98#else
99 if (sc_last == holdMod && timer_elapsed(sc_timer) < TAPPING_TERM)
100#endif
101 {
101 if (holdMod != tapMod) { 102 if (holdMod != tapMod) {
102 if (IS_MOD(holdMod)) { 103 if (IS_MOD(holdMod)) {
103 unregister_mods(MOD_BIT(holdMod)); 104 unregister_mods(MOD_BIT(holdMod));
diff --git a/quantum/process_keycode/process_space_cadet.h b/quantum/process_keycode/process_space_cadet.h
index 3ace07399..fcb70f3b4 100644
--- a/quantum/process_keycode/process_space_cadet.h
+++ b/quantum/process_keycode/process_space_cadet.h
@@ -19,6 +19,3 @@
19 19
20void perform_space_cadet(keyrecord_t *record, uint16_t sc_keycode, uint8_t holdMod, uint8_t tapMod, uint8_t keycode); 20void perform_space_cadet(keyrecord_t *record, uint16_t sc_keycode, uint8_t holdMod, uint8_t tapMod, uint8_t keycode);
21bool process_space_cadet(uint16_t keycode, keyrecord_t *record); 21bool process_space_cadet(uint16_t keycode, keyrecord_t *record);
22#ifdef NO_ACTION_TAPPING
23uint16_t get_tapping_term(uint16_t keycode, keyrecord_t *record);
24#endif
diff --git a/quantum/process_keycode/process_tap_dance.c b/quantum/process_keycode/process_tap_dance.c
index 0c7b6353e..138de0eba 100644
--- a/quantum/process_keycode/process_tap_dance.c
+++ b/quantum/process_keycode/process_tap_dance.c
@@ -14,7 +14,6 @@
14 * along with this program. If not, see <http://www.gnu.org/licenses/>. 14 * along with this program. If not, see <http://www.gnu.org/licenses/>.
15 */ 15 */
16#include "quantum.h" 16#include "quantum.h"
17#include "action_tapping.h"
18 17
19#ifndef NO_ACTION_ONESHOT 18#ifndef NO_ACTION_ONESHOT
20uint8_t get_oneshot_mods(void); 19uint8_t get_oneshot_mods(void);
@@ -167,7 +166,11 @@ void matrix_scan_tap_dance() {
167 if (action->custom_tapping_term > 0) { 166 if (action->custom_tapping_term > 0) {
168 tap_user_defined = action->custom_tapping_term; 167 tap_user_defined = action->custom_tapping_term;
169 } else { 168 } else {
169#ifdef TAPPING_TERM_PER_KEY
170 tap_user_defined = get_tapping_term(action->state.keycode, NULL); 170 tap_user_defined = get_tapping_term(action->state.keycode, NULL);
171#else
172 tap_user_defined = TAPPING_TERM;
173#endif
171 } 174 }
172 if (action->state.count && timer_elapsed(action->state.timer) > tap_user_defined) { 175 if (action->state.count && timer_elapsed(action->state.timer) > tap_user_defined) {
173 process_tap_dance_action_on_dance_finished(action); 176 process_tap_dance_action_on_dance_finished(action);