Ghost delete event before insert event

[radimitrov]

A workaround for this is to set editor.setInputType(InputType.TYPE_TEXT_VARIATION_PASSWORD); so it is related to text composition. Maybe just in the editor events?
In this example I'm simply using Gboard to type. Haven't pressed the backspace key.

Test code:

        editor.subscribeEvent(ContentChangeEvent.class, (e, unsub) -> {
            if (e.getAction() == ContentChangeEvent.ACTION_DELETE) {
                Log.d("ContentChangeEvent", "delete");
            }
            else if (e.getAction() == ContentChangeEvent.ACTION_INSERT) {
                Log.d("ContentChangeEvent", "insert");
            }
            if (e.getAction() == ContentChangeEvent.ACTION_SET_NEW_TEXT) {
                Log.d("ContentChangeEvent", "newtext");
            }
        });

Output:

2023-01-17 11:53:08.171 19029-19029/com.example.soraeditortest D/ContentChangeEvent: insert
2023-01-17 11:53:19.503 19029-19029/com.example.soraeditortest D/ContentChangeEvent: insert
2023-01-17 11:53:19.618 19029-19029/com.example.soraeditortest D/ContentChangeEvent: delete
2023-01-17 11:53:19.627 19029-19029/com.example.soraeditortest D/ContentChangeEvent: insert
2023-01-17 11:53:19.796 19029-19029/com.example.soraeditortest D/ContentChangeEvent: delete
2023-01-17 11:53:19.805 19029-19029/com.example.soraeditortest D/ContentChangeEvent: insert
2023-01-17 11:53:19.923 19029-19029/com.example.soraeditortest D/ContentChangeEvent: delete
2023-01-17 11:53:19.933 19029-19029/com.example.soraeditortest D/ContentChangeEvent: insert

[Rosemoe]

According to ducumentation for InputConnection#setComposingText:

Replace the currently composing text with the given text, and set the new cursor position. Any composing text set previously will be removed automatically.

sora-editor deletes the composing text first and inserts the new one.

[Rosemoe]

This does can be avoided in some cases (by converting the operation to a single deletion or insertion).

[radimitrov]

Didn't know Android implements composition like that, but I guess it makes it simpler.
An easy workaround on the IDE side would be a single shot timer that checks is there was an insertion for example 20ms after a deletion.
Not exactly optimal, but it will get the job done when needed. However I don't think that is a good idea to be a "fix".