Config option for debounce (maybe even debounce on based on expected search time) #85
manuelsteiner
started this conversation in
Ideas
Replies: 1 comment
-
Sounds like a good addition. Feel free to submit a PR 😄 |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hi,
after finding issue #40, I am wondering if it is a good idea to offer debouncing via an option in the configuration file or maybe even on a more fine grained basis based on dependency searches. It might make sense to distinguish between different actions in terms of expected finish time of searches.
Scenario
Consider the normal spotlight command search that filters the available commands. This is usually pretty fast and can easily be done after each keystroke. However, filtering through dependencies that might take a while to fetch from the backend takes a while and could the resulting behaviour of loading the results might not always be as smooth and could lead to unexpected behaviour while typing. This can be improved with
x-model.debounce
.Possible improvement
However, the
x-model.debounce
might not be wanted on all searches. Maybe we could introdue an attribute on dependecies to indicate if debouncing should be done or not and then conditionally usex-model
orx-model.debounce
. I haven't had a close look yetif that is easily doable.Alternatively just providing a global configuration flag to enable or disable debouncing would also be an option.
If that sounds like something you are interested in, I can work on a PR.
Beta Was this translation helpful? Give feedback.
All reactions