VisualEditor: ve.ce.SurfaceObserver.prototype.stop() called without a subsequent call to ve.ce.SurfaceObserver.prototype.start()
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	matmarex
	Apr 28 2013, 5:21 PM

Description

Something causes ve.ce.SurfaceObserver.prototype.stop() to be called without a subsequent call to ve.ce.SurfaceObserver.prototype.start().

Apparently happens after clicking on a link. I'm investigating.

Version: unspecified
Severity: normal

Status	Assigned	Task
Resolved	Jdforrester-WMF	T52781 VisualEditor: Browser blacklist issues (tracking)
Resolved	matmarex	T52813 VisualEditor: Remove Opera from blacklist
Resolved	matmarex	T38000 VisualEditor: Does not work in Opera (tracking)
Resolved	matmarex	T39861 VisualEditor on Opera 12: editing existing link doesn't work, breaks selection logic
Resolved	matmarex	T49772 VisualEditor: ve.ce.Surface.getSelectionRect() throws errors in Opera
Resolved	matmarex	T49793 VisualEditor: ve.ce.SurfaceObserver.prototype.stop() called without a subsequent call to ve.ce.SurfaceObserver.prototype.start()
Resolved	matmarex	T49794 VisualEditor: Link inspector doesn't open in Opera

• bzimport raised the priority of this task from to Lowest.Nov 22 2014, 1:17 AM

• bzimport set Reference to bz47793.

Related URL: https://gerrit.wikimedia.org/r/61358 (Gerrit Change Iaaa2a7993f3561dc663a064468c396ebfe951ce7)

Gotcha. Why does Opera have to be weird...

Change 71249 had a related patch set uploaded by Matmarex:
ve.ce.Surface: Prevent focus loss on the document node in Opera

Change 61358 abandoned by Matmarex:
Don't bind 'blur' and 'focus' on the document node in Opera

Reason:
A different approach in I7962f59bd02e0.

Change 71249 merged by jenkins-bot:
ve.ce.Surface: Prevent focus loss on the document node in Opera

Hooray! This shouldn't be causing issues anymore now.

Change 77639 had a related patch set uploaded by Matmarex:
Actually fix bug 47793 for Opera for real this time

Change 77639 merged by jenkins-bot:
Actually fix bug 47793 for Opera for real this time

Fixed a second time. Maybe. :-)

I'm hacking around mutation observers.. maybe we will get rid of SurfaceObserver soon (and get rid of those kind of problems).