Page MenuHomePhabricator

hideWhenOutOfView = true breaks popup that go out of view in RTL pages
Closed, ResolvedPublicBUG REPORT

Description

See an actual example at T281769#7174204. If a popup has hideWhenOutOfView: true (or empty, since true is the default) you're on a RTL page, and you scroll so that the widget goes out of view, when it gets back into view, the popup will be broken as the first image at T281769#7174204. I think this might be a bug in FloatableElement.computePosition.

Event Timeline

This looks like the same problem as T180173 (which also occurs in LTR).

will this be mitigated when we introduce the ocr button in the toolbar instead of as an element on top of the page? https://phabricator.wikimedia.org/T285712

will this be mitigated when we introduce the ocr button in the toolbar instead of as an element on top of the page? https://phabricator.wikimedia.org/T285712

It depends on how we're going to implement it, but that would be just for the wikisource side. The OOUI bug that this task is about needs to be fixed in OOUI itself.

matmarex claimed this task.

This seems to be working correctly after the fix for the subtask. I proposed a patch to remove the workaround in https://gerrit.wikimedia.org/r/c/mediawiki/extensions/Wikisource/+/862353.