Page MenuHomePhabricator

Newcomer tasks: panel should be behind other editing dialogs
Closed, ResolvedPublic

Description

During the newcomer's experience with guidance, there are times that other dialogs will be open along with the help panel. In those cases, we want the help panel to be behind the other dialogs. We have decided to fix this with z-indexes. Here are two examples of the order being opposite from what we want in beta:

Image selection

image.png (766×1 px, 719 KB)

Edit summary

image.png (762×1 px, 124 KB)

Event Timeline

Change 602623 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/extensions/GrowthExperiments@master] Revert "Increase help panel z-index"

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

Change 602623 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Revert "Increase help panel z-index"

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

Checked in betalabs - works as expected (also checked for arwiki betalabs and IE11/Edge browsers).
The screenshots for the cases in the task description:

Screen Shot 2020-06-05 at 1.23.28 PM.png (695×1 px, 729 KB)
Screen Shot 2020-06-05 at 1.24.19 PM.png (655×1 px, 121 KB)

An additional case with inserting a template:

Screen Shot 2020-06-05 at 1.23.59 PM.png (646×1 px, 87 KB)

And a case when another popup is present (Content translation popup):

Screen Shot 2020-06-05 at 1.27.45 PM.png (697×959 px, 111 KB)

For @RHo review
(1) VE tool bar stays in plays when a page is scrolled down, so some of the VE drop down menus might interfere with the Guidance/Help panel:

Screen Shot 2020-06-05 at 2.11.25 PM.png (638×1 px, 275 KB)

(2) LInk inspector might present the same problem as the menus from (1):
Screen Shot 2020-06-05 at 1.33.46 PM.png (638×1 px, 121 KB)

(3) the Guidance panel shifts every time when VE produces a popup:
(click for the animated gif)
shift_guidance panel.gif (642×1 px, 613 KB)

Note: To have an option to close the Guidance/Help panel from any state (the 'X' button) would be convenient.

[..]
For @RHo review
(1) VE tool bar stays in plays when a page is scrolled down, so some of the VE drop down menus might interfere with the Guidance/Help panel:

Screen Shot 2020-06-05 at 2.11.25 PM.png (638×1 px, 275 KB)

Moving back to fix so that all drop downs from the VE toolbar appear above the panel. See another example where the Bullet list is almost entirely obscured behind the help panel.

image.png (1×1 px, 337 KB)

(2) LInk inspector might present the same problem as the menus from (1):

Screen Shot 2020-06-05 at 1.33.46 PM.png (638×1 px, 121 KB)

Given that one of the easy tasks is Add a link, in my opinion we should have the add/check link popup appear over the help panel as well. See below another example with the add link pop-up almost entirely obscured by the panel.

image.png (780×1 px, 101 KB)

Moving back to Needs More Work unless @MMiller_WMF you would prefer to file separate bugs to fix [1] and [2]?

(3) the Guidance panel shifts every time when VE produces a popup:
(click for the animated gif)

shift_guidance panel.gif (642×1 px, 613 KB)

  • This is minor so I have file as a separate low bug at T255421

Note: To have an option to close the Guidance/Help panel from any state (the 'X' button) would be convenient.

Agree, the proposal to do so is on T255254

#bodyContent is its own stacking context (and within that, the VE surface too). Stacking the help panel between differents part of the editable area is not possible.

The VE toolbar is also its own stacking context, so if the help panel stacks under dropdown menus in the toolbar, it will stack under the entire toolbar, too. That would require lowering z-index to 1; not sure what other side effects that would have.

Thanks, @Tgr. Given this technical limitation, I am going to resolve this task as sufficient for our current needs. That said, this issue may come back as we build "add a link", which will involve our features pointing out and interacting closely with the visual editor.