Last modified: 2013-08-07 05:03:36 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 T49793, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 47793 - VisualEditor: ve.ce.SurfaceObserver.prototype.stop() called without a subsequent call to ve.ce.SurfaceObserver.prototype.start()
VisualEditor: ve.ce.SurfaceObserver.prototype.stop() called without a subsequ...
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
ContentEditable (Other open bugs)
unspecified
All All
: Lowest normal
: VE-deploy-2013-08-15
Assigned To: Bartosz Dziewoński
:
Depends on:
Blocks: 36000 37861
  Show dependency tree / graph
 
Reported: 2013-04-28 17:21 UTC by Bartosz Dziewoński
Modified: 2013-08-07 05:03 UTC (History)
5 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Bartosz Dziewoński 2013-04-28 17:21:27 UTC 
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.
Comment 1 Gerrit Notification Bot 2013-04-29 11:44:07 UTC 
Related URL: https://gerrit.wikimedia.org/r/61358 (Gerrit Change Iaaa2a7993f3561dc663a064468c396ebfe951ce7)
Comment 2 Bartosz Dziewoński 2013-04-29 11:44:55 UTC 
Gotcha. Why does Opera have to be weird...
Comment 3 Gerrit Notification Bot 2013-06-29 21:36:30 UTC 
Change 71249 had a related patch set uploaded by Matmarex:
ve.ce.Surface: Prevent focus loss on the document node in Opera

https://gerrit.wikimedia.org/r/71249
Comment 4 Gerrit Notification Bot 2013-06-29 21:37:08 UTC 
Change 61358 abandoned by Matmarex:
Don't bind 'blur' and 'focus' on the document node in Opera

Reason:
A different approach in I7962f59bd02e0.

https://gerrit.wikimedia.org/r/61358
Comment 5 Gerrit Notification Bot 2013-07-29 17:17:49 UTC 
Change 71249 merged by jenkins-bot:
ve.ce.Surface: Prevent focus loss on the document node in Opera

https://gerrit.wikimedia.org/r/71249
Comment 6 Bartosz Dziewoński 2013-07-29 17:21:03 UTC 
Hooray! This shouldn't be causing issues anymore now.
Comment 7 Gerrit Notification Bot 2013-08-04 21:10:20 UTC 
Change 77639 had a related patch set uploaded by Matmarex:
Actually fix bug 47793 for Opera for real this time

https://gerrit.wikimedia.org/r/77639
Comment 8 Gerrit Notification Bot 2013-08-07 03:12:58 UTC 
Change 77639 merged by jenkins-bot:
Actually fix bug 47793 for Opera for real this time

https://gerrit.wikimedia.org/r/77639
Comment 9 James Forrester 2013-08-07 03:14:22 UTC 
Fixed a second time. Maybe. :-)
Comment 10 Inez Korczyński 2013-08-07 05:03:36 UTC 
I'm hacking around mutation observers.. maybe we will get rid of SurfaceObserver soon (and get rid of those kind of problems).
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links