| Esanders | |
| Apr 12 2018, 2:32 PM |
| F17608856: Screen Shot 2018-05-02 at 8.54.15 PM.png | |
| May 3 2018, 3:55 AM |
| F17343571: image.png | |
| Apr 27 2018, 4:54 PM |
| F16935010: image.png | |
| Apr 12 2018, 2:32 PM |
| F16935013: image.png | |
| Apr 12 2018, 2:32 PM |
At small window sizes, menus from the styling toolbar such as the list menu or the more menu ("...") may expand out of the tools column. Currently they are shown below the contents as illustrated below:
When adjusting the z-index of the elements involved we need to make sure that the publish settings popup still works (and both popups are not shown at the same time):
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Place toolbar popups in default overlay | mediawiki/extensions/ContentTranslation | master | +1 -1 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Esanders | T192081 CX2: Toolbar menu appears underneath column | |||
| Resolved | Pginer-WMF | T193255 Make publish/draft settings toolbar an OOUI toolbar |
I would suggest partially reverting https://gerrit.wikimedia.org/r/#/c/413707/ and making the second toolbar an OOUI toolbar.
By Second toolbar which one you meant? Is it the publish destination as toolbar with two tools(New page, Personal draft) under that?
I'd consider this ticket a duplicate of T191508.
Note that you can't just increase the z-index of the sidebar as it is deliberately underneath the content area so that the publish destination popup works
I'd propose to have both popups not to open at the same time. Closing one popup before opening the other seems a good behaviour for transient elements like these and solves conflicts with the z-order.
Please, let me know if I'm missing something.
I don't have both popups open in either of my screenshots, so I'm not sure how that solves the problem.
By Second toolbar which one you meant? Is it the publish destination as toolbar with two tools(New page, Personal draft) under that?
Yeah, the publish destination + publish "toolbar", which is currently just two buttons. It has some visually inconsistencies with the other toolbar.
I'd consider this ticket a duplicate of T191508.
Yeah - merging in this direction though as we have more detail and discussion here.
Ok. Let me clarify. I was suggesting that you can adjust the z-index for the elements in order (content, toolbar including their popups, publish popup), and have no issues as long as both popups are not shown at the same time. Do you expect any problem with such approach?
I tried to clarify the description and provide some introduction context from the other card that has been merged as duplicate.
I was suggesting that you can adjust the z-index for the elements in order (content, toolbar including their popups, publish popup)
Do you mean dynamically change the z-index of the parent container based on whether the popup is open? If so that seems a bit hacky, and may obfuscate other stacking issues later. We should try to avoid dynamically changing z-indexes.
If you just make the second toolbar OOUI again (we already have the code in the reverted patchset), then you should be able to put all the popups in the global overlay layer trivially (thanks to T192505).
Change 429451 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/ContentTranslation@master] Place toolbar popups in default overlay
No, I was not meaning dynamically. Just making sure that popups z-index is higher than the one for both content and cards.
If you just make the second toolbar OOUI again (we already have the code in the reverted patchset), then you should be able to put all the popups in the global overlay layer trivially (thanks to T192505).
Sounds good. In general it makes sense to align with OOUI, and if it can solve the management of the popups even better. I'd like to check that our decisions here allow to support our future plans for the publish settings panel. I'll add more details in T193255.
Change 429451 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Place toolbar popups in default overlay
