Ignore expand abbreviation if multiple lines selected about codemirror6-plugin

[sergeche]

I guess you’ve used expandAbbreviation command as tab key handler? You should use abbreviation tracker instead as described in README, it contains all the required logic to handle Tab key.

The reason to expand abbreviation if range > 0 is to allow user to select specific text and expand it as abbreviation (in case if abbreviation extraction doesn’t work as expected)

[shshaw]

Hm, I currently have both, which may be part of the problem. I think I had the Tab keymap from the previous version and added abbreviationTracker when that was available.

To clarify: If you use abbreviationTracker, you do not need a separate keymap for expandAbbreviation?

[shshaw]

Okay, I did get this working with just abbreviationTracker. However, I had to set this extension to Prec.high in order for the Tab keymap to run ahead of the indentWithTab keymap

    Prec.high( abbreviationTracker( ... ) )

If you're using some of the CodeMirror default extension lists, you may need that extra Precedence bump.

[sergeche]

To clarify: If you use abbreviationTracker, you do not need a separate keymap for expandAbbreviation?

No, expandAbbreviation action should be used in case you want to expand abbreviation in some unsupported syntax like Markdown or Go (why not?). In supported syntaxes, if you want to expand by Tab, you should use abbreviation tracker since Tab key is very tricky to handle. Tracker contains all the logic to make it as smooth as possible

[shshaw]

Tricky. I think my current setup of mapping languages to certain syntaxes is sufficient for the languages we'll support. Thanks for clarifying.