elan-language / LanguageAndIDE

Apache License 2.0
3 stars 1 forks source link

Symbol Completion: include 'containing' matches from 1st char, and change case-sensitivity rule #858

Closed richardpawson closed 1 day ago

richardpawson commented 2 weeks ago

The issue with the current rules is shown with this example. Here we are in an empty field:

Although this is now basically working e.g.:

image

Now I type G and get:

image

but I add an r and the list grows.

image

I think that the list should never grow when you add more characters. This means that I think we must have both 'startsWithand 'includes' matches from the first character. Given that thestartsWith matches are listed before thecontains` matches - this is unlikely to be intrusive.

Controlled here:

image

Additionally

Change the case-sensitivity rules

If the user enters a string that contains any upper case then test against the case the user has entered. If the user's entry is all lower case then test it case insensitively.