Page MenuHomePhabricator

VisualEditor: Switching to source mode should show a spinner, pending state or similar
Closed, ResolvedPublic

Description

As of May 6th, I am experiencing a lag in switching from VE to Edit source mode, after clicking on Ok on "Switch to source editing" dialog, for 5-6 secs, it seems like nothing is happening on my browser.

This is happening even if I have perfect network speed and all other things loading properly.


Version: unspecified
Severity: enhancement

Details

Reference
bz65012

Event Timeline

bzimport raised the priority of this task from to Normal.
bzimport set Reference to bz65012.
Ryasmeen created this task.May 7 2014, 7:02 PM

This is only in beta labs, probably because beta labs is just slow. We should have a spinner and a blocking overlay, though. Right now you can even interact with the surface while we switch to source mode.

Spinner? Hmm... I've been thinking perhaps mark the switch confirmation dialog as pending and just not close it.

As for blocking overlay... Not sure about what the best way to do that is yet. I assume setting pointer-events:none; on the <body> isn't great.

Change 144505 had a related patch set uploaded by Alex Monk:
Show pending state and block clicks when switching to source mode

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

Change 144505 merged by jenkins-bot:
Show pending state on action button and keep dialog open when switching to source mode

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

Verified the fix in Betalabs

Verified the fix in test2

Verified the fix in production