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

fix(color): Rework rotary encoder acceleration calculation removing double handling #3441

Merged
merged 3 commits into from
Apr 16, 2023

Conversation

philmoz
Copy link
Collaborator

@philmoz philmoz commented Apr 6, 2023

Fixes #3428

Only tested on TX16S Mk2 so far; but this seems to work better in my testing.

@philmoz
Copy link
Collaborator Author

philmoz commented Apr 9, 2023

I've updated the PR to set the rotencSpeed variable used by the Lua getRotEncSpeed function to address issue #3378

@philmoz philmoz mentioned this pull request Apr 9, 2023
1 task
@pfeerick pfeerick added bug 🪲 Something isn't working color Related generally to color LCD radios labels Apr 9, 2023
@pfeerick pfeerick changed the title feat(color) - Rework the rotary encoder acceleration calculation, and remove double handling of acceleration. fix(color): Rework rotary encoder acceleration calculation removing double handling Apr 9, 2023
@pfeerick pfeerick added this to the 2.9 milestone Apr 10, 2023
@pfeerick pfeerick self-assigned this Apr 15, 2023
@pfeerick pfeerick linked an issue Apr 16, 2023 that may be closed by this pull request
1 task
@pfeerick pfeerick merged commit 5b0e47f into EdgeTX:main Apr 16, 2023
@philmoz philmoz deleted the rotary-accel branch October 23, 2023 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🪲 Something isn't working color Related generally to color LCD radios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scroll wheel on an X12S is difficult to operate getRotEncSpeed() doesn't work
2 participants