Disable search in multiple select field for Other Options field when not selected

[haslinghuis]

Fixes #3457

We should not be able to type here - this part is still there - but disabling also disable the search. Disabling on field exit seems to fix the issue jumping back from typing in Custom Defines to Other Options field.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[blckmn]

AUTOMERGE: (FAIL)

• github identifies PR as mergeable -> FAIL
• assigned to a milestone -> PASS
• cooling off period lapsed -> PASS
• commit count less or equal to three -> PASS
• Don't merge label NOT found -> PASS
• at least one RN: label found -> PASS
• Tested label found -> FAIL
• assigned to an approver -> PASS
• approver count at least three -> FAIL

[McGiverGim]

This disables the search? I use it a lot, each time I need to add some define.
I need to test this, but I like to have a search.

[haslinghuis]

@McGiverGim yes - but I only want to stop enabling typing in the field. Maybe Vue has a better way (as I don't want to extend with adapters).

[nerdCopter]

i also use the search, then click the filtered result.
maybe better to clear the text after click, but unsure how to clear without clearing selections. ☹️

[haslinghuis]

Agree we should not be able to type in the result, only using search.
Now you have to manually remove the text.

Also often end up typing in the options field when using custom defines field.
As we are heading to Vue components and templates not sure if we should go the route with implementation of adapters etc.

[McGiverGim]

I don't remove the text. When you search, yes, it remains in the field, but it seems is not "really" there. It seems not to affect the build, only the selected ones.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[haslinghuis]

It would be better if we could have a search box separate from the selected items as backspace will remove last selected.

[github-actions]

Do you want to test this code? Here you have an automated build:
Betaflight-Configurator-Android
Betaflight-Configurator-Linux
Betaflight-Configurator-macOS
Betaflight-Configurator-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

[nerdCopter]

It would be better if we could have a search box separate from the selected items as backspace will remove last selected.

i'd be okay with backspace deletion if the cursor could be moved with keyboard. arrow keys do not move cursor :(

edit: seems the whole UI requires mousing only. tabbing between fields is barely functional across the whole UI too. very not computer-centric.
Maybe as @GTorreil suggested. make it similar to blackbox's "Data Included" field 🤷‍♂️ (list items, do not do the 9 of 14 selected shorthand). but really i still like searching.

[nerdCopter]

fa95e911 seems to clear the typing upon field exit; approving.