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

Paddle direction parameter change #83

Merged
merged 3 commits into from
Dec 7, 2019
Merged

Conversation

VK2EFL
Copy link
Collaborator

@VK2EFL VK2EFL commented Nov 27, 2019

Added an option so that the left paddle decreases the speed/sidetone and the right paddle increases the speed/sidetone. Could be more intuitive since 'dials to the right' implies more of ...
The new option OPTION_SWAP_PADDLE_PARAMETER_CHANGE_DIRECTION controls whether this swap is in effect or not. Unless the option is defined, then the previously coded paddle directions will be implemented.
There is no increase in the compiled code size, since either way, only one piece of code is included in the final compiled code, as was the case before adding this option.

VK2EFL and others added 3 commits November 26, 2019 19:04
Change the default up/down speed change direction using the paddle press.
Modify the default code so that the left paddle decreases the speed/sidetone and the right paddle increases the speed/sidetone. Could be more intuitive since 'knobs to the right' implies more...
The new option OPTION_SWAP_PADDLE_PARAMETER_CHANGE_DIRECTION controls whether this swap is in effect or not. Unless the option is defined, then the default paddle direction will be implemented.
There is no increase in the compiled code size, since only one piece of code is included in the final compiled code, as was the case before adding this option.
@k3ng k3ng merged commit 0f54a22 into master Dec 7, 2019
k3ng added a commit that referenced this pull request Dec 7, 2019
      Resolved conflict and merged pull request 83 ( #83)
      Added OPTION_SWAP_PADDLE_PARAMETER_CHANGE_DIRECTION  to feature and settings files of all hardware profiles (OPTION from pull request 83)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants