eclipse.platform.ui icon indicating copy to clipboard operation
eclipse.platform.ui copied to clipboard

[WIP] Keep content assist popup open even if user input has no matches

Open trancexpress opened this issue 1 year ago • 4 comments

WIP, DONT MERGE

Fixes: #2260

trancexpress avatar Sep 11 '24 09:09 trancexpress

There are artifacts with the current change, will have to look into those when I find time:

gh2260_content_assist_artifacts.webm

Opened https://github.com/eclipse-jdt/eclipse.jdt.ui/issues/1643 for this.

trancexpress avatar Sep 11 '24 09:09 trancexpress

Test Results

0 files   -  1 815  0 suites   - 1 815   0s :stopwatch: - 1h 34m 29s 0 tests  -  7 697  0 :white_check_mark:  -  7 468  0 :zzz:  - 228  0 :x:  - 1  0 runs   - 24 252  0 :white_check_mark:  - 23 502  0 :zzz:  - 749  0 :x:  - 1 

Results for commit 6600fb81. ± Comparison against base commit adbbaa63.

github-actions[bot] avatar Sep 11 '24 09:09 github-actions[bot]

The proposal seems like a clear enhancement to me. I would enjoy the IDE providing this. One thing I believe is important is that if user closes the popup (hits Escape), the popup doesn't show again except if it has actual proposals.

mickaelistria avatar Sep 11 '24 09:09 mickaelistria

One thing I believe is important is that if user closes the popup (hits Escape), the popup doesn't show again except if it has actual proposals.

To be clear, https://github.com/eclipse-platform/eclipse.platform.ui/issues/2260 is for keeping the content assist popup once opened with the usual invocation. We don't aim for https://github.com/eclipse-platform/eclipse.platform.ui/issues/888, i.e. open the content assist popup automatically as soon as there is a proposal.

So we don't change how the popup is shown, we only keep it open once shown.

Though maybe I misunderstand your concern? Is there some preference that lets content assist show up automatically on typing, that I must test with?

trancexpress avatar Sep 13 '24 09:09 trancexpress