-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Change some 'RC' keywords to more appropriate 'manual_control' if also valid for joystick #23640
Change some 'RC' keywords to more appropriate 'manual_control' if also valid for joystick #23640
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for bringing this up. The wording is actually wrong.
The only thing I'd do differently is take a term e.g. "stick gesture" and then use that term everywhere and not have the one enum "manual_control" and the UI string "stick" and another enum "rc" for the same thing.
I can add a suggestion.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sfuhrer I thought "joystick gesture" is even more clear. What do you think?
Not necessarily. I associate the word "joystick" to a gamepad generally, so a source for manual control inputs not being an RC. That's also how the groundstation tab is called: |
Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
…OL_GESTURE Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
… gesture Signed-off-by: Silvan Fuhrer <silvan@auterion.com>
2dc567e
to
e7ec5f5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the add-ons @MaEtUgR !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
associate the word "joystick" to a gamepad
You're right, that would be confusing again.
Just blew my mind how inaccurate humans including me usually are with wording 😄
How's a joystick a gamepad but not the stick of a remote? Does the prefix joy change it?
Because there were gameport "joystick"s for flight simulators in the 90s?
I digress 😇
Most of all the arming/disarming reason should be "Stick gesture" and not "RC".