Page MenuHomePhabricator

[Regression wmf9] VisualEditor fails to load (ve.ui.MobileContext.css not defined?)
Closed, ResolvedPublic

Description

Open up the wikitext editor and attempt to switch to VE. It no longer works. Clicking has no effect and you remain on the wikitext editor.

On Chrome and Firefox.


Version: unspecified
Severity: critical

Event Timeline

bingle-admin wrote:

Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/AbulO05q

I'm investigating this. I know what's happening but not why.

Issue seems to be in VisualEditor
exception 'MWException' with message 'ResourceLoaderFileModule::readStyleFile: style file not found: "/srv/mediawiki/php-master/extensions/VisualEditor/lib/ve/src/ui/styles/ve.ui.MobileContext.css"' in /srv/mediawiki/php-master/includes/resourceloader/ResourceLoaderFileModule.php:902
Stack trace:

See http://bits.beta.wmflabs.org/en.wikipedia.beta.wmflabs.org/load.php?debug=false&lang=en&modules=ext.visualEditor.core.mobile%7Cext.visualEditor.mediawiki.mobile%7Cext.visualEditor.mobileViewTarget%7Cmobile.editor.ve&skin=minerva&version=20141120T011929Z&*

gerritadmin wrote:

Change 174779 had a related patch set uploaded by Jforrester:
Follow-up I7eb0d0e: ve.ui.MobileContext.css doesn't exist any more, Toto

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

gerritadmin wrote:

Change 174779 merged by jenkins-bot:
Follow-up I7eb0d0e: ve.ui.MobileContext.css doesn't exist any more, Toto

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

Fixed in master; backport scheduled for this evening's SWAT.

Checked the fix in betalabs - switching between Editor and Edit source works.

Jdforrester-WMF renamed this task from VisualEditor Mobile: [Regression wmf9] VisualEditor fails to load (ve.ui.MobileContext.css not defined?) to [Regression wmf9] VisualEditor fails to load (ve.ui.MobileContext.css not defined?).Nov 24 2014, 2:29 AM
Jdforrester-WMF set Security to None.

Still failing in wmf9, now because our fix used MobileContextOptionWidget instead of MobileContextItemWidget

Change 176052 had a related patch set uploaded (by Catrope):
Followup rEVED142a44ea38fb: fix path names to MobileContextItemWidget.{js,css}

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

Patch-For-Review

Verified the fix in Betalabs and test2

Change 176052 merged by Catrope:
Followup rEVED142a44ea38fb: fix path names to MobileContextItemWidget.{js,css}

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