Page MenuHomePhabricator

VisualEditor Mobile: [Regression] Link suggestions are not appearing while trying to add link target from link inspector in Nexus 7(Android 4.4.2) and iOS
Closed, ResolvedPublic

Description

Screenshot

Steps to reproduce:

1.Open a page with mobile VE
2.Type a text in CE
3.Select that text
4.Click on Link inspector icon from the toolbar

Observed Result:
The link inspector dialog opens but there is no suggestion for link target, also the text selected should appear in the text field inside link inspector, which is not happening as well

Device: Nexus 7(Android 4.4.2)
Environment:Betalabs

See the screenshot attached


Version: unspecified
Severity: major

Attached:

Screenshot_2014-06-17-15-38-38.png (800×1 px, 40 KB)

Details

Reference
bz66753

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 3:20 AM
bzimport set Reference to bz66753.

This is not android specific issue, its happening for iOS as well.
Also, it has gone to production now.So, the environment is not just Betalabs now :(

I am getting this issue intermittently now. Surely the behavior got changed/improved which is why it is not reproducible anymore in a straight forward way.

I will see if I can find any consistent step that ensures this issue to happen.

Found the steps.
This happens every second attempt on adding a link where the keyboard gets hidden after adding the first link in CE

So the steps are:

1.Open a page with mobile VE
2.Type a text in CE
3.Select that text
4.Click on Link inspector icon from the toolbar
5.Select a suggestion that appears in the dropdown box
6.After this link gets added successfully, hide the keyboard
7.Again type a new text and try to add a link to it from the link inspector

Observed Result:
No link suggestions appear, cannot close the link inspector.

Change 143524 had a related patch set uploaded by Catrope:
Use globalOverlay as the contextOverlay in MobileContext

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

Change 143524 merged by jenkins-bot:
Use globalOverlay as the contextOverlay in MobileContext

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

Verified the fix in Betalabs with iOS

Verified the fix in Betalabs with Android as well

Verified the fix in test2 with iOS and Android.

Verified the fix in production en.wiki for iOS only, for Android there is a different issue, where the link text is not appearing inside the link inspector, so cannot follow the exact steps to reproduce to check it there.