Pasting content results in scroll flickering
Closed, ResolvedPublic8 Estimated Story Points
Actions

Assigned To

Authored By

	Krinkle
	Feb 16 2015, 1:10 AM

Description

Edit medium to long document. (mediawiki.org/OOjs)
Paste plain text content from an another web page. (github:wikimedia/oojs#readme)

Unexpected:
Page scrolls, causing some kind of distorted pain rendering, and moves back to (almost) the same position, showing the pasted contents.

capture.mov1 MBDownload

Details

	Subject	Repo	Branch	Lines +/-
	Restore scroll immediately after paste	VisualEditor/VisualEditor	master	+2 -1

Customize query in gerrit

Related Objects

Mentioned In: rGVEDc16da3031e1e: Restore scroll immediately after paste

Event Timeline

Krinkle created this task.Feb 16 2015, 1:10 AM

Krinkle raised the priority of this task from to Needs Triage.

Krinkle updated the task description. (Show Details)

Krinkle added projects: VisualEditor-CopyPaste, VisualEditor-ContentEditable, VisualEditor-Performance, Performance Issue.

Krinkle subscribed.

Restricted Application added a project: VisualEditor. · View Herald TranscriptFeb 16 2015, 1:10 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Jdforrester-WMF triaged this task as Medium priority.Mar 2 2015, 4:01 AM

Jdforrester-WMF set Security to None.

Jdforrester-WMF moved this task from To Triage to Freezer on the VisualEditor board.Mar 2 2015, 4:14 PM

Jdforrester-WMF added a project: VisualEditor 2014/15 Q4 blockers.Apr 8 2015, 7:31 PM

Jdforrester-WMF moved this task from Nominated to Backlog on the VisualEditor 2014/15 Q4 blockers board.

Jdforrester-WMF edited a custom field.

@Krinkle, from quick testing this looks like it's fixed for me – can you confirm?

Hasn't changed much. It still scrolls to the top leaving the user nowhere near the content that was affected or inserted having to scroll back down manually to reveal the pasted content. The only difference I saw is that, aside from jumping to the top, it now also briefly animates down a few pixels.

https://www.mediawiki.org/w/index.php?title=OOjs&veaction=edit
Cursor at start of Compatibility paragraph.
Return to start new paragraph under this heading.
Copy text (e.g. "OOjs is optimised for modern ECMAScript 5 environments. However, care has been taken to maintain parser compatibility with ES3 engines (such as for IE 6-8).").
Paste text.

Expected:
Scroll offset unchanged.

Actual:
Page scrolls back to the very top of the page, and then animates down a few pixels to the top of the editor surface.

New capture:

Untitled.mov3 MBDownload

Note that I'm manually scrolling from the top of the article back down to where the text is pasted, that's not done by the software.

Jdforrester-WMF added a project: Epic.Jun 2 2015, 8:52 PM

Jdforrester-WMF edited projects, added VisualEditor 2015/16 Q1 blockers; removed VisualEditor 2014/15 Q4 blockers.Jun 18 2015, 4:21 PM

Jdforrester-WMF merged a task: T103558: Pasting stuff while editing makes the page jump to the top while using VE.Jun 23 2015, 6:41 PM

Jdforrester-WMF added a subscriber: Niharika.

• Elitre subscribed.Jun 24 2015, 3:22 PM

Jdforrester-WMF moved this task from Nominated to Backlog: T1 on the VisualEditor 2015/16 Q1 blockers board.Jun 30 2015, 7:23 PM

Jdforrester-WMF moved this task from Freezer to TR1: Releases on the VisualEditor board.Jul 21 2015, 6:21 PM