Page MenuHomePhabricator

VisualEditor page options menu goes off-screen in RTL wikis
Closed, ResolvedPublic8 Estimated Story Points

Description

The Page options menu, which appears when the "hamburger" button is clicked, goes off-screen in RTL wikis.

Screenshot with wrong display in Hebrew:
https://commons.wikimedia.org/wiki/File:VisualEditorBug2a.png

Screenshot with correct display in English:
https://commons.wikimedia.org/wiki/File:VisualEditorBug2b.png

(I have a vague recollection that I reported it once, but I cannot find it now. If somebody does find it, please mark this one as a dupe.)

Event Timeline

Amire80 raised the priority of this task from to Needs Triage.
Amire80 updated the task description. (Show Details)
Amire80 added projects: VisualEditor, I18n.
Amire80 changed Security from none to None.

Change 179565 had a related patch set uploaded (by Amire80):
Add noflip to oo-ui-popupToolGroup-left and -right

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

Patch-For-Review

Setting appropriate left and right values for oo-ui-popupToolGroup-left and oo-ui-popupToolGroup-right by adding noflip seems to fix the issue, but because I'm not an experienced oojs-ui dev, this needs review.

For now I fixed this locally in he.wikipedia by adding:

.oo-ui-popupToolGroup-left > .oo-ui-toolGroup-tools {
	left: 0;
	right: auto;
}
matmarex added a subscriber: matmarex.

The real issue is that ClippableElement doesn't handle clipping with the left edge of browser window, only with the right edge.

Change 179581 had a related patch set uploaded (by Bartosz Dziewoński):
[WIP] ClippableElement: Handle clipping with left edge

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

Patch-For-Review

Change 179565 abandoned by Amire80:
Add noflip to oo-ui-popupToolGroup-left and -right

Reason:
Done better in I0c2d9800a1c0bdbe523c978307546ec205202681 .

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

Change 179581 merged by jenkins-bot:
ClippableElement: Handle clipping with left edge

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

My fix for this caused part of T98506.