Page MenuHomePhabricator

Allow putting OOUI toolbar menus into an overlay
Closed, ResolvedPublic

Description

To allow toolbar menus to escape confines of stacking contexts and avoid overlap issues like this:

…we have to allow them to be put into an overlay.

Also tagging VE because its floating toolbar probably complicates this. I have to find out if we need to support a combination of both these features.

Event Timeline

matmarex triaged this task as Medium priority.Apr 18 2018, 9:51 PM
matmarex created this task.
matmarex raised the priority of this task from Medium to High.Apr 18 2018, 9:51 PM
matmarex edited projects, added VisualEditor (Current work); removed VisualEditor.

Change 428136 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[oojs/ui@master] Toolbar: Put all popups (from PopupToolGroup and PopupTool) into an overlay

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

@matmarex The changed behavior is confusing. Sure, stacking context might get unwanted results like the one in the screenshot, but are we sure about this?!

There seems to be a lot of weird behaviour to arise from this right now

@Volker_E Looks like you were testing OOUI with the broken 'overflow: hidden' change that was merged and reverted today (https://gerrit.wikimedia.org/r/#/c/428616/) – try again on master, the menus should not be cut off like that.

@Ryasmeen We need to make sure, that this works across themes, in products like Flow and CX2.

Volker_E edited projects, added OOUI (OOUI-0.26.5); removed OOUI.

Change 428136 merged by jenkins-bot:
[oojs/ui@master] Toolbar: Put all popups (from PopupToolGroup and PopupTool) into an overlay

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

Volker_E removed a project: Patch-For-Review.
Volker_E removed a subscriber: gerritbot.

Putting this on resolved, any issues found should be filed as follow-ups.