Last modified: 2014-03-12 21:19:03 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T59034, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57034 - VisualEditor: SurfaceObserver sets selection to start of document after closing reference dialog in Chrome
VisualEditor: SurfaceObserver sets selection to start of document after closi...
Status: RESOLVED DUPLICATE of bug 58090
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: High normal
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-13 22:21 UTC by Roan Kattouw
Modified: 2014-03-12 21:19 UTC (History)
5 users (show)

See Also:
Web browser: Google Chrome
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Roan Kattouw 2013-11-13 22:21:57 UTC
1. Edit an existing reference or insert a new one
2. When the dialog is closed, the reference is selected and a callout is displayed
3. 200ms later the selection moves to the start of the document

Somehow, for some reason, SurfaceObserver is "fixing" the selection here.
Comment 1 D Chan 2013-11-14 02:26:51 UTC
Happens on Chromium 25 but not on Firefox 24 (on my Ubuntu system).

When the reference is inserted, the CE cursor gets reset to the start (as always happens with a ContentEditable when you change the content). But then the CE cursor updates the DM cursor, whereas it should happen the other way round.

Evidence: Make SurfaceObserver.pollOnceInternal into a no-op, then right-cursor to a word and click Reference, Insert Reference. Then the CE cursor gets reset to the start of the document, whereas the DM cursor is still in the place where the reference appears.
Comment 2 Roan Kattouw 2014-03-12 21:19:03 UTC

*** This bug has been marked as a duplicate of bug 58090 ***

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links