Feature: Add support for Xbox gamepad controls - #18825
Conversation
There was a problem hiding this comment.
I would recommend adding the below controls.
Left trigger > Left click
Right trigger > Right click
Hold left trigger + Left stick > Click and drag
If the context menu/flyout is open, B should close it and not go back a folder.
Using the mouse, no hover event's are triggered and highlights/information isn't visible
Files is goes into an unrecoverable crash where I have to restart the system, this seem to be when anything in the omnibar has it's drop down open
0x5bfa
left a comment
There was a problem hiding this comment.
Use CsWin32 instead of manually defining PInvokes. Also currently the helper does nothing?
|
@Josh65-2201 Should I add |
|
I don't see a reason to remove the existing ones if they don't cause conflicts |
|
Thanks for the clarification |
Resolved / Related Issues
To prevent extra work, all changes to the Files codebase must link to an approved issue marked as
Ready to build. Please insert the issue number following the hashtag with the issue number that this Pull Request resolves.Steps used to test these changes
Stability is a top priority for Files and all changes are required to go through testing before being merged into the repo. Please include a list of steps that you used to test this PR.