Skip to content

Rotary encoder movement not to close throttle warning#1408

Merged
pfeerick merged 1 commit intoEdgeTX:mainfrom
piotrva:throttle-warning-exit-fix
Jan 11, 2022
Merged

Rotary encoder movement not to close throttle warning#1408
pfeerick merged 1 commit intoEdgeTX:mainfrom
piotrva:throttle-warning-exit-fix

Conversation

@piotrva
Copy link
Contributor

@piotrva piotrva commented Jan 10, 2022

Fixes #1407

Summary of changes:
Checking only key presses for closing throttle warning instead of any event.
This replicates the behavior for switch warning.

Copy link
Member

@pfeerick pfeerick left a comment

Choose a reason for hiding this comment

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

Nice catch! Would have been an issue for other radios with rotary encoders also (i.e. X9D+2019 was also affected)

@pfeerick pfeerick added this to the 2.6 milestone Jan 11, 2022
@pfeerick pfeerick added B&W Related generally to black and white LCD radios bug 🪲 Something isn't working labels Jan 11, 2022
@pfeerick pfeerick merged commit 192ca87 into EdgeTX:main Jan 11, 2022
@piotrva piotrva deleted the throttle-warning-exit-fix branch January 11, 2022 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B&W Related generally to black and white LCD radios bug 🪲 Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Moving scroll wheel closes throttle warning

2 participants