Page MenuHomePhabricator

Hover highlights not updated when document modified
Closed, ResolvedPublic

Description

Steps to reproduce:

  1. Open a page with VE that has some block nodes in it.
  2. Place your cursor in a paragraph a block node
  3. Mouse over the block node
  4. Press return or backspace

Observe that the light blue highlight stays in the previous position but the block node moves with the cursor.
If you now, click on that blue highlight, it will show the selection highlight along with context menu in that position.

Screen Shot 2018-09-07 at 4.13.34 PM.png (567×1 px, 127 KB)

Screen Shot 2018-09-07 at 4.13.57 PM.png (472×1 px, 116 KB)

Screen Shot 2018-09-07 at 4.33.25 PM.png (364×1 px, 64 KB)

Event Timeline

Deskana moved this task from To Triage to Needs Discussion/Analysis on the VisualEditor board.

Possibly the same as, or related to, T93660? I also could've sworn I saw a bug almost exactly like this related to moving infoboxes around, but I can't find that right now...

Needs evaluation by engineers.

Esanders renamed this task from Weird highlight behavior for all block nodes to Highlights not updated when document modified.Sep 12 2018, 12:20 PM
Esanders updated the task description. (Show Details)
Esanders renamed this task from Highlights not updated when document modified to Hover highlights not updated when document modified.Sep 12 2018, 12:35 PM

Change 460009 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] FocusableNode: Start listening to position events even if not focused

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

Change 460009 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] FocusableNode: Start listening to position events even if not focused

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

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

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

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

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