Show Other Results: Delayed doesn't work if "Reset search after:" is disabled

quicksilver / Quicksilver

Quicksilver Project Source

http://qsapp.com

Apache License 2.0

2.73k stars 285 forks source link

Show Other Results: Delayed doesn't work if "Reset search after:" is disabled #2753

Closed pjrobertson closed 2 years ago

pjrobertson commented 2 years ago

Set up prefs as below:
Search, and see how the results are displayed immediately.

Expected behaviour: results are delayed when displaying.

Relevant code is: QSSearchObjectView.m:L854 - the time to display the results list uses the reseat search after value. It should have it's own preferences, or just be hard-coded e.g. as 0.5s for simplicity

pjrobertson commented 2 years ago

Scratch that. There already is a defaults pref Reset Delay - I just had it set as 0s. The default of 2s in QSDefaults.plist is probably a bit too long though.

skurfer commented 2 years ago

After a lot of experimenting, I settled on 1.1s for myself. I can’t imagine there’s a value that would make everyone happy, but maybe that’s a better default?

pjrobertson commented 2 years ago

Yeah, 1.1s sounds more reasonable to me. I don't want to give people who've been using the default 2s a scare if we make it too short, so I'll change it to 1.5s in main now