Page MenuHomePhabricator

VE toolbar popups are left aligned for right aligned tools
Closed, ResolvedPublic

Assigned To
Authored By
Esanders
Jun 5 2023, 1:21 PM
Referenced Files
F37132606: Screenshot 2023-07-08 at 00.08.58.png
Jul 7 2023, 11:13 PM
F37132604: Screenshot 2023-07-08 at 00.08.20.png
Jul 7 2023, 11:13 PM
F37099479: Screenshot 2023-06-09 at 20.44.38.png
Jun 9 2023, 7:53 PM
F37099477: Screenshot 2023-06-09 at 20.44.13.png
Jun 9 2023, 7:53 PM
F37099471: Screenshot 2023-06-09 at 20.43.57.png
Jun 9 2023, 7:53 PM
F37099468: Screenshot 2023-06-09 at 20.43.30.png
Jun 9 2023, 7:53 PM
F37094214: image.png
Jun 5 2023, 1:21 PM
F37094211: image.png
Jun 5 2023, 1:21 PM

Description

In full width Vector there usually wasn't room in the viewport for the popups to left align, but now we are fixed width there often is.

We should tell the toolbar to prefer right alignment of popups for right aligned tools.

Observed:

image.png (256×1 px, 72 KB)

Expected:

image.png (253×836 px, 46 KB)

Left aligned tools should still have a left aligned popup:

image.png (344×836 px, 118 KB)

Event Timeline

Change 927183 had a related patch set uploaded (by Esanders; author: Esanders):

[oojs/ui@master] Match alignment of toolbar popups with the position of the tool

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

Change 927183 merged by jenkins-bot:

[oojs/ui@master] Match alignment of toolbar popups with the position of the tool

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

@Esanders This is still left aligned. Am I missing some testing steps?

Screenshot 2023-06-09 at 20.43.30.png (2×3 px, 1001 KB)

Screenshot 2023-06-09 at 20.43.57.png (2×3 px, 563 KB)

Screenshot 2023-06-09 at 20.44.13.png (2×3 px, 461 KB)

Screenshot 2023-06-09 at 20.44.38.png (2×3 px, 493 KB)

Testing on https://en.wikipedia.beta.wmflabs.org/,

  • open VE
  • interract with the toolbar and toggle width

Do I need to switch anything on/off?

matmarex removed a project: Editing QA.

It's waiting for an OOUI release. Sorry, that's my bad, I didn't look which code repository this change was in.

It's waiting for an OOUI release. Sorry, that's my bad, I didn't look which code repository this change was in.

Makes sense. I'll come back to it then.

Change 935793 had a related patch set uploaded (by Catrope; author: Catrope):

[mediawiki/core@master] Update OOUI to v0.47.2

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

Change 935793 merged by jenkins-bot:

[mediawiki/core@master] Update OOUI to v0.47.3

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

@Esanders This is still left aligned. Am I missing some testing steps?

Screenshot 2023-06-09 at 20.43.30.png (2×3 px, 1001 KB)

Screenshot 2023-06-09 at 20.43.57.png (2×3 px, 563 KB)

Screenshot 2023-06-09 at 20.44.13.png (2×3 px, 461 KB)

Screenshot 2023-06-09 at 20.44.38.png (2×3 px, 493 KB)

Testing on https://en.wikipedia.beta.wmflabs.org/,

  • open VE
  • interract with the toolbar and toggle width

Do I need to switch anything on/off?

✅ toolbar popups are right-aligned for right aligned tools and left-aligned for left-aligned tools

Screenshot 2023-07-08 at 00.08.20.png (1×2 px, 712 KB)

Screenshot 2023-07-08 at 00.08.58.png (1×2 px, 693 KB)

ppelberg claimed this task.

Test wiki on Patch demo by ESanders (WMF) using patch(es) linked to this task was deleted:

https://patchdemo.wmflabs.org/wikis/17bd15c7b9/w/