Page MenuHomePhabricator

Inspector not working correctly on double annotation
Closed, ResolvedPublic1 Estimated Story Points

Description

Create a double annotation:

<a href="foo"><span lang="en">Bar</span></a>

Notice that clicking in the middle of the text only shows one inspector:

image.png (123×446 px, 18 KB)

Event Timeline

Esanders updated the task description. (Show Details)
Esanders set the point value for this task to 1.

Change 439613 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Follow-up I125969a: Replace $.closest with $.parents + $.addBack

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

Change 439613 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Follow-up I125969a: Replace $.closest with $.parents + $.addBack

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

Change 439624 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (8231b4d88)

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

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

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

It still shows only one inspector when I put cursor at the very beginning or at the very end. Shouldn't it behave the same way for those positions?

Screen Shot 2018-06-12 at 8.35.52 PM.png (524×841 px, 87 KB)

Screen Shot 2018-06-12 at 8.35.31 PM.png (515×615 px, 42 KB)

matmarex triaged this task as Unbreak Now! priority.Jun 14 2018, 10:58 PM

Per Dan on T197213, this is unbreak-now. We should probably backport the patch.

It still shows only one inspector when I put cursor at the very beginning or at the very end. Shouldn't it behave the same way for those positions?

I think this is the correct behavior. The link context should only display if we're displaying the active link styling (the light blue background and border). You might need to use keyboard arrow keys to navigate "inside" the link in such cases.

This is deployed everywhere already (since the train this morning).

Vvjjkkii renamed this task from Inspector not working correctly on double annotation to h9aaaaaaaa.Jul 1 2018, 1:04 AM
Vvjjkkii reopened this task as Open.
Vvjjkkii removed Esanders as the assignee of this task.
Vvjjkkii lowered the priority of this task from Unbreak Now! to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed the point value for this task.
Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot renamed this task from h9aaaaaaaa to Inspector not working correctly on double annotation.Jul 2 2018, 12:37 PM
CommunityTechBot closed this task as Resolved.
CommunityTechBot assigned this task to Esanders.
CommunityTechBot raised the priority of this task from High to Unbreak Now!.
CommunityTechBot set the point value for this task to 1.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added subscribers: gerritbot, Aklapper.