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

Re-fix Mousekey Movements #5740

Merged
merged 2 commits into from
May 6, 2019
Merged

Conversation

drashna
Copy link
Member

@drashna drashna commented Apr 30, 2019

After the new movement model was instroduced, it broke diagonal momement, again. Reapplying fix from #3147 to both old and new acceleration method.

Types of Changes

  • Core
  • Bugfix

Issues Fixed or Closed by This PR

  • Issue reported by Ergodox EZ users

Checklist

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING document.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.
tmk_core/common/mousekey.c Outdated Show resolved Hide resolved
tmk_core/common/mousekey.c Outdated Show resolved Hide resolved
tmk_core/common/mousekey.c Outdated Show resolved Hide resolved
tmk_core/common/mousekey.c Outdated Show resolved Hide resolved
@vomindoraan
Copy link
Contributor

I can confirm that this fixes the diagonal movement issue. Tested on a ChibiOS board.

Co-Authored-By: drashna <drashna@live.com>
Copy link
Contributor

@vomindoraan vomindoraan left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@mechmerlin mechmerlin left a comment

Choose a reason for hiding this comment

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

Thanks!

@mechmerlin mechmerlin merged commit 8a9c90a into qmk:master May 6, 2019
@drashna drashna deleted the fix/qmk_mouse_movement branch May 7, 2019 06:08
drashna added a commit to zsa/qmk_firmware that referenced this pull request May 7, 2019
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
KauyonKais pushed a commit to KauyonKais/qmk_firmware that referenced this pull request May 8, 2019
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
KauyonKais pushed a commit to KauyonKais/qmk_firmware that referenced this pull request May 8, 2019
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
fdidron added a commit to zsa/qmk_firmware that referenced this pull request May 20, 2019
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
* Re-fix Mousekey Movements

After the new movement model was instroduced, it broke diagonal momement, again.  Reapplying fix from qmk#3147 to both old and new acceleration method.

* Make diagonal mouse report checks more readable

Co-Authored-By: drashna <drashna@live.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants