1543312 - caret and characters insertion are erratic in a contenteditable with tinyMCE on Android

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect, P3)

Description

[Karl Dubost💡 :karlcow]

This is https://webcompat.com/issues/17764

1. With Firefox 68 Nightly on Android
2. Go to https://minecraft.curseforge.com/projects/better-builders-wands
3. sign in (top right menu. Creating an account with twitch is free)
4. scroll down the page until reaching the comment section.
5. click on Reply on one of the comment.
6. Notice that the caret is set automatically on the line after the In reply to…
7. Request the focus to get the keyboard
8. Type a couple of characters

Expected:
The text is added like in Chrome on Android.

Actual:
The caret is jumping to the first character of the first line.
Any subsequent character input happens in random order.

This is NOT reproducible on

• Chrome Android,
• Firefox Desktop,
• Firefox Desktop + Firefox Android UA + RDM.

Comment 1

[Makoto Kato [:m_kato]]

I guess that GeckoEditable doesn't insert text to valid position. This is GeckoEditable in Android, not editor issue.

Comment 2

[Firefox Bug Husbandry Bot]

Migrating Webcompat whiteboard priorities to project flags. See bug 1547409.

Comment 3

[Firefox Bug Husbandry Bot]

See bug 1547409. Migrating whiteboard priority tags to program flags.

Comment 4

[Makoto Kato [:m_kato]]

This might not Android issue. GekocEditable listens selection change via editor, but no received. When debugging desktop too, selection change isn't fired to widget. Root cause is editor? But I cannot create easy test case yet...

Comment 5

[Makoto Kato [:m_kato]]

Attachment: sample

Comment 6

[Makoto Kato [:m_kato]]

After editor is attached, if pres shell is re-created, this issue occurs. editor already attaches to selection, but selection is re-created by new pres shell. So editor must attach new selection. (or tear down editor on destroying pres shell)

Comment 7

[Makoto Kato [:m_kato]]

Attachment: Bug 1543312 - Tear down editor when PresShell is destroyed.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66e8aafac348
Tear down editor when PresShell is destroyed. r=masayuki

Comment 9

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/66e8aafac348

Comment 10

[Ryan VanderMeulen [:RyanVM]]

I think we can let this ride the trains given that it's not a new issue.