Skip to content

[FEATURE REQUEST] UI Support for Multiple (And Composite) Bindings on an Action #21

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

Open
Valafi opened this issue Jan 26, 2024 · 1 comment

Comments

@Valafi
Copy link

Valafi commented Jan 26, 2024

Is your feature request related to a problem? Please describe.
I'm making an autocomplete mod for the terminal. I have my "Autocomplete" action set to the "tab" key. Pressing it repeatedly cycles through the autocomplete list.

I then decided to add a composite binding of "ctrl+tab" to the same action with a processor of "invert" so that the autocomplete list can be cycled in reverse. Here I ran into a problem where the default binding was getting triggered before my composite binding could be (I was getting a value of 1 instead of -1). The only way I found I could fix it was to override the default binding with an empty path, and re-add it after the composite binding.

At this point, I realized that the UI only shows the default binding created by the API (which is now blank). It does not show my composite binding or re-added binding.

Describe the solution you'd like
Ideally, the UI could handle showing all of the bindings for actions (including composites).

Describe alternatives you've considered
I considered putting the composite binding on a separate action, but that's almost the same issue, because the UI is still only showing the default binding here too.

Additional context
I understand this is a pretty large request, so I apologize in advance if you decide to tackle it!

@Rune580
Copy link
Owner

Rune580 commented Jan 27, 2024

I'll have to think of a good way to represent composite bindings in UI. On the topic of the default binds being prioritized, going to have to look into it more to see if I can figure out a solution for that too.

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

No branches or pull requests

2 participants