Bug 496087 - 0 ms delay for CA auto activation
Summary: 0 ms delay for CA auto activation
Status: NEW
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: PDT (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: PHP Core CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 433298 467353 558967
Blocks:
  Show dependency tree
 
Reported: 2016-06-14 05:49 EDT by Kaloyan Raev CLA
Modified: 2020-05-14 10:16 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kaloyan Raev CLA 2016-06-14 05:49:11 EDT
Currently we have 200 ms delay for the code assist auto activation. I find it a little bit annoying and feel more comfortable when setting the delay to 0 ms.

We've done a lot of performance improvements. This 200 ms is noticeable and may be unexpected by users (including me).

I believe that the auto activation delay had its role in the past when computers were slow and excessive code assist ruined the typing experience. This is not the case anymore nowadays.

Note that JDT already switched to 0 ms delay.

I think it is a good moment to do this change early in the release cycle.
Comment 1 Eclipse Genie CLA 2016-06-14 06:17:17 EDT
New Gerrit change created: https://git.eclipse.org/r/75225
Comment 2 Michal Niewrzal CLA 2016-06-14 06:20:01 EDT
We should be careful with this option :) We did many performance improvements but I'm not sure if enough. But I agree that now it's the best time to test it.
Comment 3 Dawid Pakula CLA 2016-06-14 06:21:00 EDT
JDT have auto activation delay = 0 together with activation chars (especially after ".").

On larger projects (many types), PDT can often froze UI. Especially for instance creation (exception is slowest), method calls etc... For those we still need delay (try s| on symfony project or z| on zend project).

But 0 delay for sequences like "->", "::" will help a lot !
Comment 4 Kaloyan Raev CLA 2016-06-14 06:37:09 EDT
Can we have bugs opened for all cases where 0 ms delay causes troubles? And mark these bugs as blocking this one.

It does not make sense to apply this change, if we know it will cause troubles. But it would be helpful to know what issue needs to be resolved.

I've got feedback from people switching from Zend Studio to alternative products because of the performance. Some of the pointed the noticeable delay in code assist as the main issue. Hence, I really want to get rid of this delay :)
Comment 5 Dawid Pakula CLA 2016-06-14 07:32:38 EDT
Safer will be help with bug 251156 (bug 458335 for PDT) and calculating completion outside UI thread ;)

Anyway, I'm adding potential blockers.
Comment 6 Dawid Pakula CLA 2018-05-23 17:18:40 EDT
PDT 6.0 introduces, to make 0ms delay for assist, we need small modification in Platform.

In auto-activated mode, selection should be empty (as in netbeans, VS code etc..). Without this user (or just me :P) will be angry if accidentaly use enter without esc.