Page MenuHomePhabricator

'Change label' in link inspector broken when cursor is at end of link
Closed, ResolvedPublic

Description

The root cause is T221967, but we don't have a good fix for that at the moment.

As reported in T221309.

Event Timeline

Change 516524 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Make annotationsAtModelSelection fail in fewer cases

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

Change 516524 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Make annotationsAtModelSelection fail in fewer cases

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

Change 517137 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (44556e8fd)

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

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

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

Change 520127 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[VisualEditor/VisualEditor@master] ve.ui.AnnotationContextItem: Use ve.ce.Surface's logic for detecting annotations

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

The workaround fixes this for all cases except where the link consists of just one character. I filed a separate task about this: T227241 and moved the patch https://gerrit.wikimedia.org/r/520127 there. Please ignore that problem in QA, we'll get back to it :)

ppelberg claimed this task.