Page MenuHomePhabricator

Changing a link inside a table cell makes the page re-scroll unnecessarily
Closed, ResolvedPublic1 Story Points

Description

In Chrome, Win 8, Vector;
I'm trying to change the target for the "language variant" link at the bottom of https://www.mediawiki.org/wiki/VisualEditor#History .
Right after I select the new target, the page scrolls to the top.

This doesn't take place if you insert or alter a node (template, comment), apply/unapply a simple annotation (bold), or apply/alter the other rich annotation (language).

Event Timeline

Elitre created this task.Mar 8 2016, 3:17 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 8 2016, 3:17 PM
Elitre added a comment.Mar 8 2016, 4:32 PM

In Opera, it does jump up, but then refocuses again immediately after that.

Jdforrester-WMF renamed this task from Changing a link inside a table makes the page jump to Changing a link inside a table cell makes the page re-scroll unnecessarily.Mar 8 2016, 4:32 PM
Jdforrester-WMF triaged this task as Normal priority.
Jdforrester-WMF updated the task description. (Show Details)
Jdforrester-WMF set the point value for this task to 1.

As the inspector opens it sets the selection to null which cancels the table editing mode. Doesn't happen in the core demo though.

Esanders added a subscriber: dchan.Mar 8 2016, 4:45 PM

Change 275845 had a related patch set uploaded (by Esanders):
Link inspector: Remove premature call to onLinkTypeIndexSet

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

Change 275845 merged by jenkins-bot:
Link inspector: Remove premature call to onLinkTypeIndexSet

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

Jdforrester-WMF closed this task as Resolved.Mar 8 2016, 5:27 PM
Jdforrester-WMF assigned this task to Esanders.
Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.
Elitre awarded a token.Mar 8 2016, 5:29 PM