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

Arming vehicle using action via joystick controller is not requiring slide to confirm #832

Open
JoaoMario109 opened this issue Mar 19, 2024 · 2 comments
Labels
enhancement New feature or request ux Has to do with the user experience

Comments

@JoaoMario109
Copy link
Contributor

When using the controller to perform actions that require confirmation as arm, disarm, etc... The confirmation pop-up isn't appearing. Now that we've implemented global enable/disable for slide to confirm lib in #761 and added the ability to confirm via the controller in #822 we need to ensure that all actions requiring confirmation, such as arming and disarming, work with the controller, not just through the UI.

@JoaoMario109 JoaoMario109 added enhancement New feature or request ux Has to do with the user experience labels Mar 19, 2024
@ES-Alexander
Copy link
Contributor

Hmm, I don't think I agree with this.

Joystick buttons can be made intentionally hard to press accidentally (by physical design), and can also be composite actions already (by software configuration), in which case they shouldn't always need an additional confirmation step that requires removing hands from the joystick (e.g. like the "emergency stop" button on machinery, or using a modifier key as the confirmation of intent).

We should have a discussion on the desired behaviour here, because "take your hands off the joystick" doesn't seem like a practical default to me. If implementing nuanced behaviour is also impractical then my personal preference would be to encourage good practices when configuring the joystick profiles, rather than requiring confirmations off the joystick.

@rafaellehmkuhl
Copy link
Member

Just to confirm, the agreed solution to this is to allow the user to add the "hold to confirm" functionality to some actions, right?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux Has to do with the user experience
Projects
None yet
Development

No branches or pull requests

3 participants