Page MenuHomePhabricator

Add a switch-back-to-WT call-out in VE if coming from switching
Closed, ResolvedPublic1 Estimated Story Points

Description

Provide the same control as we do in the wikitext editor.

"You have switched to visual editing"
"You can switch back to source editing at any time by clicking on this icon."

Event Timeline

Like an education popup on the switch button?

Like an education popup on the switch button?

With different behaviour; shown always if not seen before and the user switched into this edit. Will need to deal with the other pop-ups (edit notices) I'm afraid.

With different behaviour; shown always if not seen before and

Aren't the education popups shown if not seen before? What's different with the "always" here?

Will need to deal with the other pop-ups (edit notices) I'm afraid.

Meaning that if we open another one, this popup has to disappear?

With different behaviour; shown always if not seen before and

Aren't the education popups shown if not seen before? What's different with the "always" here?

The blue dots are shown. The pop-ups themselves aren't. This isn't a blue dot, but an immediate pull-out (the same way it works in WT, and the same way that edit notices work in VE).

Will need to deal with the other pop-ups (edit notices) I'm afraid.

Meaning that if we open another one, this popup has to disappear?

Yeah. Sorry for the mess.

the same way that edit notices work

Oh yeah, that hack. It'll work for this, but:

Will need to deal with the other pop-ups (edit notices) I'm afraid.

Meaning that if we open another one, this popup has to disappear?

Yeah. Sorry for the mess.

We'll need to figure out what to do when we want to show both edit notices and this.

<Krenair> When you open VE, it can open both the edit notices popup and this new popup
<James_F> Yeah.
<Krenair> but we don't want to show both at the same time
<Krenair> so which one gets priority?
<James_F> This one.
<Krenair> I suppose suppressing edit notices in this case isn't so bad as the user presumably just saw them on the wikitext editor

Change 283120 had a related patch set uploaded (by Alex Monk):
[WIP] Add a popup after switching into VE from WTE

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

Change 283120 merged by jenkins-bot:
Add a popup after switching into VE from WTE

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