Page MenuHomePhabricator

Inconsistencies around textbox highlighting in link inspector
Closed, ResolvedPublic1 Estimated Story Points


Steps to reproduce:

  1. Open VisualEditor on some page.
  2. Highlight some text.
  3. Open link inspector
  4. Now switch from "Search Pages" to "External link" tab, observe that the highlight around text box is red
  5. Now switch back to "Search Pages"
  6. Switch back to "External link", this time the highlight around the text box is blue.

Event Timeline

Deskana triaged this task as Medium priority.Aug 1 2017, 7:07 PM
Deskana moved this task from To Triage to TR1: Releases on the VisualEditor board.
Deskana set the point value for this task to 1.

It seems to always be blue for me. Are you still seeing this, or...?

I can't get this to work with these exact reproduction steps, but if you change step 1 to "Highlight some text and open link inspector" then this can be reproduced. I'll change the description.

Ah, I think my own "try selecting something" phase accidentally hit some text which was a valid URL, explaining my problems.

Change 374563 had a related patch set uploaded (by DLynch; owner: DLynch):
[mediawiki/extensions/VisualEditor@master] MWLinkAnnotationInspector: check input validity when switching modes

Change 374563 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] MWLinkAnnotationInspector: check input validity when switching modes

Deskana assigned this task to DLynch.