The toolbar for VE is appearing outside the border of the page
Compare the screenshots below:
Beta Cluster:
Production:
Ryasmeen | |
May 1 2020, 9:07 PM |
F31790358: Screen Shot 2020-05-01 at 2.06.55 PM.png | |
May 1 2020, 9:07 PM |
F31790347: Screen Shot 2020-05-01 at 2.02.51 PM.png | |
May 1 2020, 9:07 PM |
The toolbar for VE is appearing outside the border of the page
Compare the screenshots below:
Beta Cluster:
Production:
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Jdlrobson | T16501 A way to hide the sidebar providing a "fullscreen" view | |||
Resolved | ovasileva | T247032 [EPIC] Build collapsible sidebar and deploy to all test wikis | |||
Resolved | ovasileva | T258072 [REQUEST] QA Support on the new header | |||
Resolved | ovasileva | T250375 QA VE with new header and collapsible sidebar | |||
Resolved | ppelberg | T251648 [Regression pre-wmf.31] The toolbar for VE is appearing outside the border of the page |
This is caused by the Vector changes, it seems; the bug goes away if you disable them ("Use Legacy Vector" in skin preferences).
The rule removed from the modern layout is:
https://gerrit.wikimedia.org/r/plugins/gitiles/mediawiki/skins/Vector/+/f6cbbbfd957b9c9639049455f252190923f07a12/resources/skins.vector.styles/legacy/layout.less#89
Only viewport width >=982px is affected.
Modern will have a new layout, part of which this rule was dropped (atm it seems the padding won't depend on screen width).
The new layout is only active on the beta cluster yet, live servers are not affected.
To resolve this there are a few options:
Change 593954 had a related patch set uploaded (by Aron Manning; owner: Aron Manning):
[mediawiki/skins/Vector@master] [modern] Temporarily restore the #content (.mw-body) padding for VisualEditor's alignment
Change 593955 had a related patch set uploaded (by Aron Manning; owner: Aron Manning):
[mediawiki/skins/Vector@master] Add CSS class on body to mark the skin version
we have no plans for this right now. If the editing team can let us know how they want to deal with this (for example would Aron's suggested classes help) I can then prioritise this and make sure it's addressed. My preference would be to remove the CSS rules from VisualEditor and/or update them to be more future proofed. This will likely break again when we introduce the fixed width.
I defer to Olga to whether this blocks deployment.
I think we're okay for deployment as we won't be deploying to any of the test wikis until fixed width is completed. That said, we will be going to office wiki - @ppelberg, @iamjessklein - is this okay with you? We will be QAing the changes to VE and making fixes as a part of the fixed width work that is a part of the first test wiki deployment T250375: QA VE with new header and collapsible sidebar.
skin-vector-2 probably makes sense going forward, although we should use it as little as possible
cc @matmarex
It isn't ideal, but such is the nature of a shared beta cluster. Is there a way to switch back to Vector-1 using a query string parameter. That way if we (or other teams) want to conduct users tests with a more live-like environment we can do this?
switch back to Vector-1 using a query string parameter.
https://en.wikipedia.beta.wmflabs.org/wiki/Spain?useskinversion=1
https://en.wikipedia.beta.wmflabs.org/wiki/Spain?useskinversion=2
Change 593954 abandoned by Bartosz Dziewoński:
[modern] Temporarily restore the #content (.mw-body) padding for VisualEditor's alignment
Reason:
Let's do https://gerrit.wikimedia.org/r/593955 instead, I'll submit a patch in VE to use that
Change 594793 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[mediawiki/extensions/VisualEditor@master] Update toolbar styles for Vector changes
Thank you for checking, @ovasileva. +1 to what @Esanders commented: so long as the following conditions are met (which it seems they are), we're fine with this being deployed to http://office.wikimedia.org/ [1]:
- Olga, you mentioned "office wikis"; I assumed the plural was a typo. If this is not the case, and if deployed right now, this issue would affect other wikis beyond http://office.wikimedia.org/ please let me know.
Yup, it was a typo. Fixed now!
Change 593955 merged by jenkins-bot:
[mediawiki/skins/Vector@master] Add CSS class on body to mark the skin version
I think editing team can handle this for now so moving out of sprint board. Let me know if you need any more assistance from us!
Change 595202 had a related patch set uploaded (by Aron Manning; owner: Aron Manning):
[mediawiki/extensions/VisualEditor@master] Fix editor toolbar positioning for Vector modern layout
Change 594793 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor@master] Update toolbar styles for Vector changes