-
-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
Use the correct keycode when updating WPM #19599
Merged
Merged
Conversation
This file contains 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
drashna
approved these changes
Jan 17, 2023
fauxpark
approved these changes
Jan 17, 2023
freznel10
added a commit
to freznel10/qmk_firmware
that referenced
this pull request
Jan 18, 2023
commit f2b75cbe06e49bf941c5b6dfa481b9caf255edde Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Jan 17 07:30:05 2023 -0700 rebase and cleanup commit 3ce417eca4883c17016af529cc803bc04e7f7f00 Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Jan 17 07:18:10 2023 -0700 finally fix hscroll fully and add maybe ability to change multiplier at runtime commit a6a62b76719b577999bd4609b4c52b5117086adc Author: Alabastard-64 <snipac+github@gmail.com> Date: Mon Jan 16 13:44:14 2023 -0700 major update/revision most issues fixed commit b7a90fb940d7c7fa2d952793f98c2984aed5f915 Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Jan 10 22:54:48 2023 -0700 bugfix for disable function commit 756275b4208e9c4710416d6fc9cb7e36492be551 Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Jan 10 21:32:56 2023 -0700 Fix HID descriptor for Horizontal scroll, add disable function, and various bugfixes commit 671c176a7aaa5812f9391b083bcfed86fc93239d Author: Alabastard-64 <snipac+github@gmail.com> Date: Sat Jan 7 22:55:18 2023 -0700 linting except on vusb and doc update commit 83c477cff9e42389365c9aeb32c29d028b49c21e Author: Alabastard-64 <snipac+github@gmail.com> Date: Sat Jan 7 21:49:47 2023 -0700 all tested versions working and doc commit d8590eee684f57f82e5e60fc5eff518524e460b4 Author: Alabastard-64 <snipac+github@gmail.com> Date: Fri Jan 6 11:53:00 2023 -0700 real initial release everything working and tested on Atmel U4 chips commit f23af60faa8ea2509ff0c03f83fac471e58ef9b6 Author: Alabastard-64 <snipac+github@gmail.com> Date: Wed Jan 4 12:15:44 2023 -0700 update: not broken... but not working either commit 4b58f08c206a1b29635890f4b36aeea9e4818e65 Author: Alabastard-64 <snipac+github@gmail.com> Date: Mon Jan 2 09:51:37 2023 -0700 initial commit (currently broken...) commit ebada5092192550303c1f2e7caa4d82cfcb70a53 Author: Alabastard-64 <snipac+github@gmail.com> Date: Fri Dec 23 14:58:39 2022 -0700 doc update commit 825eea16f51e19b781fca51a53b32b091431c55e Author: Alabastard-64 <snipac+github@gmail.com> Date: Fri Dec 23 14:51:37 2022 -0700 linting commit b5d37d6153a483974c047d712a7e34ce1960bf34 Author: Alabastard-64 <snipac+github@gmail.com> Date: Thu Dec 22 19:13:24 2022 -0700 fixes and doc improvements commit c9ec67ed252cc8a310e4ce0dfa5b540978eae214 Author: Alabastard-64 <snipac+github@gmail.com> Date: Wed Dec 21 23:07:52 2022 -0700 fix previous changes to precision mode now working as intended additional mode fixes commit c6282586b07f2f17f891e6ef4b2ab980e3d23d63 Author: Alabastard-64 <snipac+github@gmail.com> Date: Wed Dec 21 20:29:11 2022 -0700 doc update and precision changes commit 414bff0f3a79637975a5c1926dc59cd652a3a8eb Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Dec 13 00:53:47 2022 -0700 more doc updates/code cleanup commit 77444eda136ddf6e7c395221a263039763c7f3cd Author: Alabastard-64 <snipac+github@gmail.com> Date: Tue Dec 13 00:20:15 2022 -0700 code and doc fixup commit d89945415449ba5a06f8892b1bf9771cf18ad9aa Author: Alabastard-64 <snipac+github@gmail.com> Date: Mon Dec 12 23:10:19 2022 -0700 update and linting commit c13accd651e4fd1bf2754aba00ecf82f350e6210 Author: Alabastard-64 <snipac+github@gmail.com> Date: Sun Dec 11 21:08:08 2022 -0700 post rebase/squash cleanup commit ce34c0aa810607cd347ad536c9eae2b7cfd6180e Author: Alabastard-64 <96358682+Alabastard-64@users.noreply.github.com> Date: Tue Aug 30 01:23:41 2022 -0600 parent c6ab70e author Alabastard-64 <96358682+Alabastard-64@users.noreply.github.com> 1661844221 -0600 committer Alabastard-64 <snipac+github@gmail.com> 1670816485 -0700 parent c6ab70e author Alabastard-64 <96358682+Alabastard-64@users.noreply.github.com> 1661844221 -0600 committer Alabastard-64 <snipac+github@gmail.com> 1670813789 -0700 parent c6ab70e author Alabastard-64 <96358682+Alabastard-64@users.noreply.github.com> 1661844221 -0600 committer Alabastard-64 <snipac+github@gmail.com> 1670812067 -0700 Squashed all previous Commits commit d448ff4 Merge: 3b60a05 625e574 Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 07:40:37 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 625e574 Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 23:39:09 2023 -0800 Format code according to conventions (qmk#19615) commit 3b60a05 Merge: 275673c 9f84b9a Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 05:57:55 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 9f84b9a Author: Ivan Gromov <38141348+key10iq@users.noreply.github.com> Date: Tue Jan 17 09:57:18 2023 +0400 [Keyboard] Add kt3700 (qmk#19432) Co-authored-by: Sergey Vlasov <sigprof@gmail.com> Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Ryan <fauxpark@gmail.com> commit 275673c Merge: 69397d4 c6cc104 Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 05:57:09 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit c6cc104 Author: JW2586 <70282382+JW2586@users.noreply.github.com> Date: Tue Jan 17 05:56:37 2023 +0000 [Keyboard] Waterfowl - Updated default keymap (qmk#19438) Co-authored-by: jack <0x6a73@protonmail.com> commit 69397d4 Merge: db7ca07 7b795b2 Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 05:49:24 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 7b795b2 Author: Alabahuy <rezzaadio@gmail.com> Date: Tue Jan 17 12:48:44 2023 +0700 [Keyboard] add kamigakushi (qmk#19514) commit db7ca07 Author: Jouke Witteveen <j.witteveen@gmail.com> Date: Tue Jan 17 06:23:57 2023 +0100 Process Tap Dances before WPM/Velocikey (qmk#19599) commit a52b3aa Merge: 2cf904b 49f3ffa Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 04:34:24 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 49f3ffa Author: d-floe <31980883+d-floe@users.noreply.github.com> Date: Mon Jan 16 22:33:49 2023 -0600 [Keyboard] Add Bubble 75 (qmk#18863) Co-authored-by: Drashna Jaelre <drashna@live.com> Co-authored-by: Ryan <fauxpark@gmail.com> commit 2cf904b Merge: 0ff5292 4098ff5 Author: QMK Bot <hello@qmk.fm> Date: Tue Jan 17 01:55:46 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 4098ff5 Author: era <73109780+eerraa@users.noreply.github.com> Date: Tue Jan 17 10:55:13 2023 +0900 [Keyboard] Add ERA65 (qmk#19591) Co-authored-by: Ryan <fauxpark@gmail.com> commit 0ff5292 Author: Albert Y <76888457+filterpaper@users.noreply.github.com> Date: Tue Jan 17 09:54:51 2023 +0800 Refactor pixel rain animation (qmk#19606) commit 7acc3f4 Author: Ryan <fauxpark@gmail.com> Date: Tue Jan 17 12:54:26 2023 +1100 ChibiOS: Consolidate report sending code (qmk#19607) commit 6fd4d8c Merge: 4c7f67f 465b6a1 Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 22:42:25 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 465b6a1 Author: Tom Barnes <tom@mechboards.co.uk> Date: Mon Jan 16 22:41:52 2023 +0000 Docs/space b cleanup (qmk#19612) * Clean up remaining mentions of the deprecated Space B reset method * cleanup auto_tag.yml that got sucked in commit 4c7f67f Merge: b6e42db 2bff00e Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 09:33:19 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 2bff00e Author: Pablo Martínez <58857054+elpekenin@users.noreply.github.com> Date: Mon Jan 16 10:32:43 2023 +0100 Small doc changes (qmk#19601) commit b6e42db Merge: b077e63 917d93e Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 08:52:56 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 917d93e Author: Sergey Vlasov <sigprof@gmail.com> Date: Mon Jan 16 11:52:18 2023 +0300 [Docs] Fix `JOYSTICK_AXIS_COUNT` name in docs (qmk#19605) commit b077e63 Merge: 401b475 4f83b67 Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 08:49:16 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 4f83b67 Author: adiabatic <adiabatic@users.noreply.github.com> Date: Mon Jan 16 00:48:41 2023 -0800 [Keymap] Improve Zweihander layout for the Ergodox EZ (qmk#18737) commit 401b475 Merge: 955829b fe27e46 Author: QMK Bot <hello@qmk.fm> Date: Mon Jan 16 00:56:04 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit fe27e46 Author: Ryan Baker <76887060+ryanbaekr@users.noreply.github.com> Date: Sun Jan 15 19:55:30 2023 -0500 [Keyboard] Add rb87 (qmk#19546) commit 955829b Author: Sergey Vlasov <sigprof@gmail.com> Date: Mon Jan 16 02:29:29 2023 +0300 Fix joystick build for ChibiOS (qmk#19602) `joystick_report_t` was renamed to `report_joystick_t`, but apparently one place in the code was missed. commit 0730740 Author: Joel Challis <git@zvecr.com> Date: Sun Jan 15 15:04:32 2023 +0000 Add mmoskal/uf2-stm32f103 bootloader support (qmk#19594) commit a920714 Author: Ryan <fauxpark@gmail.com> Date: Sun Jan 15 19:40:52 2023 +1100 usb_main.c: remove `CH_KERNEL_MAJOR` check (qmk#19597) commit 8046467 Merge: d40516c 44bcde1 Author: QMK Bot <hello@qmk.fm> Date: Sun Jan 15 08:03:56 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 44bcde1 Author: Christian Brauner <brauner@kernel.org> Date: Sun Jan 15 09:03:27 2023 +0100 [Keymap] Update brauner preonic layout (qmk#19595) commit 2c98e79 Author: jack <0x6A73@pm.me> Date: Sun Jan 15 01:03:16 2023 -0700 [Keyboard] Fixup gingko65 matrix pins (qmk#19589) commit d40516c Merge: 5f46e29 e7d1b21 Author: QMK Bot <hello@qmk.fm> Date: Sun Jan 15 03:58:42 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit e7d1b21 Author: jurassic73 <50930136+jurassic73@users.noreply.github.com> Date: Sat Jan 14 19:58:09 2023 -0800 [Keyboard] Update handwired/split89 to new standard. (qmk#19540) Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com> commit 5f46e29 Merge: 45851a1 4a53d9c Author: QMK Bot <hello@qmk.fm> Date: Sun Jan 15 03:47:13 2023 +0000 Merge remote-tracking branch 'origin/master' into develop commit 4a53d9c Author: Bastien BALOUP <35816948+Bastien-Baloup@users.noreply.github.com> Date: Sun Jan 15 04:46:13 2023 +0100 [Keymap] Add Bastian-Baloup sofle keymap (qmk#18836) Co-authored-by: Ryan <fauxpark@gmail.com>
omikronik
pushed a commit
to omikronik/qmk_firmware
that referenced
this pull request
Jan 22, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
This is a leftover from #17935. I kept the velocikey updates next to the wpm updates since they are similar in spirit.
Moving the tap dance preprocessing up ensures that WPM is calculated for the correct keycode, since the code can change as a result of tap dance preprocessing.
Types of Changes
Issues Fixed or Closed by This PR
There is no serious bug here, this change is mainly for being 'correct'.
Checklist