Page MenuHomePhabricator

Error `Cannot read property 'getScrollContainer' of null` when saving on mobile
Closed, ResolvedPublic

Description

The following error occurs after saving on mobile VE:

VM31 jquery.js:3818 jQuery.Deferred exception: Cannot read property 'getScrollContainer' of null TypeError: Cannot read property 'getScrollContainer' of null
    at VeUiMWSurface.ve.ui.Surface.toggleMobileGlobalOverlay (http://dev.wiki.local.wmftest.net:8080/w/extensions/VisualEditor/lib/ve/src/ui/ve.ui.Surface.js?88f5b:608:36)
    at http://dev.wiki.local.wmftest.net:8080/w/extensions/VisualEditor/lib/ve/src/ui/ve.ui.Surface.js?88f5b:587:16
    at fire (http://dev.wiki.local.wmftest.net:8080/w/resources/lib/jquery/jquery.js?6a07d:3268:31)
    at Object.fireWith [as resolveWith] (http://dev.wiki.local.wmftest.net:8080/w/resources/lib/jquery/jquery.js?6a07d:3398:7)
    at Object.deferred.(anonymous function) [as resolve] (http://dev.wiki.local.wmftest.net:8080/w/resources/lib/jquery/jquery.js?6a07d:3736:36)
    at http://dev.wiki.local.wmftest.net:8080/w/resources/lib/ooui/oojs-ui-windows.js?1ebec:1522:21
    at mightThrow (http://dev.wiki.local.wmftest.net:8080/w/resources/lib/jquery/jquery.js?6a07d:3534:29)
    at process (http://dev.wiki.local.wmftest.net:8080/w/resources/lib/jquery/jquery.js?6a07d:3602:12) undefined

Because it's in a Deferred, it doesn't actually interfere with anything.

Event Timeline

DLynch created this task.Apr 23 2019, 4:45 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptApr 23 2019, 4:45 PM

Change 505822 had a related patch set uploaded (by DLynch; owner: DLynch):
[VisualEditor/VisualEditor@master] toggleMobileGlobalOverlay: check whether the target exists before using it

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

Change 505822 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] toggleMobileGlobalOverlay: check whether the target exists before using it

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

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

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

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

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

ppelberg closed this task as Resolved.Jun 18 2019, 12:26 AM