From 41885c4e72d18dfdbdc7a53682b6f1186e057d46 Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Tue, 3 Nov 2020 16:05:12 -0500 Subject: [PATCH] Revert company-idle-delay to package default (0.5) Setting a default for this variable is dangerous. Set it to low and modes with slow backends will give users the impression that this is typing latency (they'll suspect something else, likely Doom, before suspecting their lsp servers or company backends). Set it too high and users may misinterpret this as latency. Turn it off (set to nil) and I'll get a dozen "code completion isn't working" bug reports. I've settled on company's own default of 0.5 (which is twice as slow as Doom's original default) because responsiveness while typing is more important (and more frustrating to deal with), but it's still enabled, so users are more likely to notice it than assume code completion isn't working. --- modules/completion/company/config.el | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/modules/completion/company/config.el b/modules/completion/company/config.el index 055491639..109b9861b 100644 --- a/modules/completion/company/config.el +++ b/modules/completion/company/config.el @@ -4,8 +4,7 @@ :commands company-complete-common company-manual-begin company-grab-line :hook (doom-first-input . global-company-mode) :init - (setq company-idle-delay 0.25 - company-minimum-prefix-length 2 + (setq company-minimum-prefix-length 2 company-tooltip-limit 14 company-tooltip-align-annotations t company-require-match 'never