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

Clear in Keymapping should say 'Clear chosen controller' #898

Open
dfilskov opened this issue May 3, 2024 · 3 comments
Open

Clear in Keymapping should say 'Clear chosen controller' #898

dfilskov opened this issue May 3, 2024 · 3 comments

Comments

@dfilskov
Copy link

dfilskov commented May 3, 2024

In Key-button remapping please rename 'Hold Menu -> Clear' to 'Hold Menu -> Clear chosen controller'.

Or make holding the Menu button clear re-mappings for all controllers / all PIDs.

At least it should say 'No controller chosen' when you open the Key/button remap screen and instantly hold Menu before tapping a controller and it has no PID to clear. Otherwise it seems to work when it actually doesn't.

Also it could say 'Clearing controller' instead or just 'Clearing'.

I couldn't make fire work on any joysticks on the C64 core no matter what I did.

I had to delete the C64_input_*.map files (in config/input) to make Fire work again.

I remapped the buttons (even tapped ESC to 'clear button mapping') and cleared any key<->button mappings by holding the Menu shortcut button. Reset play assignment. Nothing worked. No way to clear the wrong mappings it seems.

@dfilskov
Copy link
Author

dfilskov commented May 3, 2024

Here are the files I had in config/inputs.

keymaps_couldn't_be_cleared.zip

@rsn8887
Copy link

rsn8887 commented Sep 6, 2024

I found out how it works:
In the button/key remap dialog, you first have to press some button on the controller that you want to clear, THEN hold menu button, or press ESC twice, to clear! Then it works. The reason is that MiSTer needs to know the PID/VID of the controller you want to clear before it can clear it.

@dfilskov
Copy link
Author

dfilskov commented Sep 6, 2024

The reason is that MiSTer needs to know the PID/VID of the controller you want to clear before it can clear it.

Ah I see. Good work! - Thank you!

So 'Clear' does not mean 'Clear all remappings' but rather 'Clear chosen controller'.

I'll update the bug report title.

@dfilskov dfilskov changed the title Keymapping can't be cleared (e.g. C64) Clear in Keymapping should say 'Clear chosen controller' Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants