Page MenuHomePhabricator

Popup not necessarily positioned correctly when tabbing through links
Open, LowPublic

Description

If I tab onto a link, the hover card always appears above the link, even if there isn't room to show it.

Hovering with the mouse results in the card being placed below the link if necessary.

image.png (203×420 px, 59 KB)

QA Results - Beta

ACStatusDetails
1T270215#7366016

Event Timeline

ovasileva lowered the priority of this task from Medium to Low.Feb 22 2021, 1:44 PM
Jdlrobson moved this task from Triaged but Future to unsed on the Web-Team-Backlog board.
Jdlrobson added a subscriber: ovasileva.
Jdlrobson subscribed.

I tested this on Chrome, and now I see the page preview always appearing under the link so I think this is fixed?

Screen Shot 2021-08-12 at 10.09.44 AM.png (696×1 px, 645 KB)

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

AC1: When tabbing through a page's links. Popups should appear above the link towards the bottom of the page, and below the link towards the top of the page.

Screen Recording 2021-09-13 at 6.59.25 AM.mov.gif (938×1 px, 1 MB)

Edtadros subscribed.

Test Result - Beta (Retest)

Status: ❌ FAIL
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

❌ A{F34648841}C1: When tabbing through a page's links. Popups should appear above the link towards the bottom of the page, and below the link towards the top of the page.

While testing Prod I noticed that the behavior changed if I scrolled down (or some other reason, I'm not sure).

The initial tabbing worked for the links on the top:

Screen Recording 2021-09-20 at 9.07.18 AM.mov.gif (792×1 px, 2 MB)

But if I scrolled down using the mouse then clicked, or if I tabbed to the point where the page scrolled down then shift-tabbed back up to a link near the top, the popup would still appear above the link.

Screen Recording 2021-09-20 at 9.08.27 AM.mov.gif (792×1 px, 2 MB)

Edtadros updated the task description. (Show Details)

Given the error is still present, I'm throwing back to the backlog for another day since we've wrapped up our timebox for page previews work.

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

[mediawiki/extensions/Popups@master] Account for scroll offset when using non-mouse positioning

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

Change #1013281 merged by jenkins-bot:

[mediawiki/extensions/Popups@master] Account for scroll offset when using non-mouse positioning

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

matmarex assigned this task to Esanders.
matmarex removed a project: Patch-For-Review.
matmarex subscribed.
Jdlrobson reassigned this task from Esanders to Edtadros.

This needs QA.