Page MenuHomePhabricator

jQuery.IME not working in VisualEditor/2017 wikitext editor
Closed, ResolvedPublic8 Estimated Story PointsBUG REPORT

Description

ULS not working properly for Kannada in Visual editor. It shows the Kannada character and English characcter for every key pressed. We don't get the desired Kannada words.

Steps to Reproduce:

  1. Open http://kn.wikipedia.org
  2. Click on ನನ್ನ ಪ್ರಯೋಗಪುಟ or open your SandBox
  3. Click on ಸಂಪಾದಿಸಿ (invoke Visual Editor)
  4. Select any Kannada keyboard layout from ULS, for ex., KGP/Nudi/KP Rao
  5. Type any word in Kannada, for ex., knfnw

Already tested page http://kn.wikipedia.org/wiki/ಸದಸ್ಯ:Pavanaja/ನನ್ನ_ಪ್ರಯೋಗಪುಟ
Actual Results:
The result is ಕkನnfನnಡw.

Expected Results:
But it should have been ಕನ್ನಡ

ULS Screenshot from Visual Editor.png (479×1 px, 91 KB)

ULS Screenshot from source editor.png (537×1 px, 81 KB)

The same issue is also reported for Telugu Wikipedia.

Event Timeline

I can't set the priority of this task due to a bug.

@dchan Can you take a look at this please? The impact may be severe, but I don't know enough this to know that myself.

I also can't set the story points of this task due to a bug. Cool.

It is probably severe, because it's also reported for Telugu.

Hindi language also same issue

It is probably severe, because it's also reported for Telugu.

I think the problem is there for all Indic Wikis. I checked with Tulu (it uses Kannada IME, hence it has to be there), Telugu and Malayalam. For every key pressed, it displays the English character and the Indic character.

Elitre renamed this task from ULS not working properly for Kannada in Visual editor to ULS not working properly for Indic wikis in Visual editor.Sep 20 2017, 1:55 PM

This seems happening in 2017 wikitext editor too.

Elitre renamed this task from ULS not working properly for Indic wikis in Visual editor to ULS not working properly for Indic wikis in visual editor/2017 wikitext editor.Oct 3 2017, 7:27 AM

ULS not working properly for Kannada in Visual editor. It shows the Kannada character and English characcter for every key pressed. We don't get the desired Kannada words.

A quick note that not all of ULS is broken, but specifically input methods. Language and font selection seem to be ok.

Catrope triaged this task as Unbreak Now! priority.Oct 3 2017, 5:44 PM

Is this a regression? Do we know when it was last working as that will help us pin down the problem?

Pinging @dchan

I first observed ULS malfunction on sept 16 or 15, before that it was working fine

Jdforrester-WMF set the point value for this task to 8.
dchan renamed this task from ULS not working properly for Indic wikis in visual editor/2017 wikitext editor to jQuery.IME not working in VisualEditor/2017 wikitext editor.Oct 7 2017, 12:14 PM

This was introduced in 37d0c27478983c9607cd85fdc6551b16a9bbb3a5 - an exception is thrown whenever jQuery.IME attempts to do any text replacement, and the timing of the exception means that the document ends up with both the substitution and the typed character text.

It happens because ve.ce.Surface#onDocumentInput uses event.originalEvent.inputType, but jQuery.IME creates synthetic events with $element.trigger( 'input' ) whose originalEvent is undefined.

Change 382971 had a related patch set uploaded (by Divec; owner: Divec):
[VisualEditor/VisualEditor@master] Don't assume events have an originalEvent object

https://gerrit.wikimedia.org/r/382971

Change 382971 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Don't assume events have an originalEvent object

https://gerrit.wikimedia.org/r/382971

Change 383253 had a related patch set uploaded (by Catrope; owner: Catrope):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (c5c5123)

https://gerrit.wikimedia.org/r/383253

Change 383253 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (c5c5123)

https://gerrit.wikimedia.org/r/383253

Should now be fixed in master.

@Deskana This should be deploy to Production and tested.

@Deskana Still facing same issue, not solved yet

@Deskana This should be deploy to Production and tested.

The fixes were tested by @Jdforrester-WMF, and will also soon be tested by @Ryasmeen. They will be automatically deployed over the next two days.

In T176104#3672506, @Anooprao wrote:

@Deskana Still facing same issue, not solved yet

The fixes are now in the code base, but not live on the site yet. As mentioned above, they will be deployed in two days.

@Deskana fix is live now, thanks for resolving.

@Anooprao Glad it's working. :-)