Replies: 1 comment 3 replies
-
Hey @Foggy2, Thanks for bringing this up! The way hooks and custom actions interact has always been a bit tricky - especially when you only want to override hook actions for specific buttons. I really like your approach, however I'd go for a bit of a different take on the per-event input boolean:
This would give users the flexibility to disable hook actions per button while also allowing them to fully deactivate a button if needed. Also, huge thanks for offering to implement this - would love to see your contribution! 🚀 What do you think? Curious to hear what the community thinks about this too. 🔥 |
Beta Was this translation helpful? Give feedback.
3 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
Hi All,
I am starting this discussion to continue what has started in #693.
I understand that the current behavior of controllers is to simultaneously fire both the hook action and any custom actions, when a button is pressed.
I can see situations where this behavior is beneficial, but I can also see situations where this behavior is undesirable.
For example, I could see a scenario such as the following being encountered commonly. When using two button controllers such as the Ikea E2201 Rodret. The light hook for those controllers, contains an action mapping for all possible button combinations by default.
If you wish to re-map a button to perform a different action, such as using a double press event to quickly recall a preset color/brightness combination. You are unable as the double press will also trigger the hook action of adjusting the light color which will cause a conflict.
To resolve this potential conflict. I propose that another configuration option (toggle switch) be added to the blueprint UI that allows the user to select the behavior of custom actions. With the possible choices being:
I am happy to take on the project of updating all of the controllers if it is decided to adopt this added functionality.
Happy to discuss, let me know your thoughts.
Thanks!
Beta Was this translation helpful? Give feedback.
All reactions