Regression: Nested references do not open if user clicks on [ or ] (which are wrapped in span)
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Jdlrobson
	Jul 15 2020, 7:12 PM

Description

Related: T243822

Visit https://en.m.wikipedia.beta.wmflabs.org/wiki/Nested_references
Click [note 1]
click [1]
Repeat.

Expected:
The drawer updates

Actual:
It works the first time, but not the second time.

Screen Shot 2020-07-15 at 12.11.52 PM.png (796×1 px, 120 KB)

Screen Shot 2020-07-15 at 12.12.21 PM.png (1×2 px, 1 MB)

Developer notes:

The underlying HTML has changed exposing a flaw in the click handler - this means clicking the [ or ] doesn't open the nested reference:

Before:

<sup id="cite_ref-2" class="reference"><a href="#cite_note-2">[note 1]</a></sup>

Now:

<sup id="cite_ref-2" class="reference"><a href="#cite_note-2"><span>[</span>note 1<span>]</span></a></sup>

Acceptance criteria

test is skipped
MobileFrontend is patched
Test is no longer skipped in Minerva

QA Results - Beta

AC	Status	Details
1	✅	T258096#6434480

QA Results - Prod

AC	Status	Details
1	✅	T258096#6434894

QA Results - Prod (Sept 10, confirmed `1.36.0-wmf.8`)

AC	Status	Details
1	✅	T258096#6452585

Details

Subject	Repo	Branch	Lines +/-
QA: Restore nested reference test	mediawiki/skins/MinervaNeue	master	+1 -1
Harden nested references click handler against different markup	mediawiki/extensions/MobileFrontend	master	+13 -8
Selenium: Skip nested reference test.	mediawiki/skins/MinervaNeue	master	+1 -1
Nested references should not trigger console warning	mediawiki/skins/MinervaNeue	master	+5 -0

Customize query in gerrit

Related Objects

Mentioned In: T258528: Minerva browser tests failing without a proper summary
Mentioned Here: T258728: MobileFrontend browser test failing: User:<username> "before all" hook – Can't call setValue on element with selector "#wpName1" because element wasn't found
T243822: Reference -Preview and -Popup shown both with Minerva in desktop mode

Event Timeline

Jdlrobson created this task.Jul 15 2020, 7:12 PM

Restricted Application added subscribers: Masumrezarock100, Aklapper. · View Herald TranscriptJul 15 2020, 7:12 PM

Jdlrobson assigned this task to ovasileva.Jul 21 2020, 12:30 AM

Jdlrobson added a project: Web-Team-Backlog.

Jdlrobson moved this task from Incoming to Needs Prioritization on the Web-Team-Backlog board.

This is a recent regression and is causing various browser test failures.

I believe this relates to Reference Previews - both Popups previews and MobileFrontend are conflicting with one another - only one of those should be applied. We might need to sink with WMDE and resolve T243822 to handle this one.

Change 615298 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Nested references should not trigger console warning

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

gerritbot added a project: Patch-For-Review.Jul 21 2020, 8:25 PM

Jdlrobson moved this task from Needs Analysis to Upcoming on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.Jul 22 2020, 12:14 AM

Jdlrobson renamed this task from Nested references do not always open on MInerva to Browser test failure: Nested references do not always open.Jul 23 2020, 7:49 PM

Blocked on T258728

Change 615298 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Nested references should not trigger console warning

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

ReleaseTaggerBot added a project: MW-1.36-notes (1.36.0-wmf.2; 2020-07-28).Jul 24 2020, 11:00 PM

Maintenance_bot removed a project: Patch-For-Review.Jul 24 2020, 11:10 PM

Jdlrobson reassigned this task from ovasileva to Edtadros.Jul 28 2020, 6:42 PM

Jdlrobson added a subscriber: ovasileva.

Change 616891 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] Fix faulty nested references click handler

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

gerritbot added a project: Patch-For-Review.Jul 28 2020, 7:04 PM

Jdlrobson renamed this task from Browser test failure: Nested references do not always open to Regression: Nested references do not open if user clicks on [ or ] (which are wrapped in span).Jul 28 2020, 7:06 PM

Jdlrobson updated the task description. (Show Details)

Change 616899 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] QA: References clicks should apply to the sup element not span

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

Thanks to @Edtadros we've skipped the test, so the browser test build is green again, but there is a real regression here we should fix caused by some underlying parser changes in I2cc88069b19e7611f23c83ca993f9caa70f786f0

I'd be tempted to just skip this and move on as it looks like it will be tricky to debug but on the other hand it is a useful test for something we seldom check.

Jdlrobson removed a project: Reference Previews.Jul 28 2020, 10:02 PM

Jdlrobson added a project: MobileFrontend.

Jdlrobson claimed this task.Jul 29 2020, 7:39 PM

Jdlrobson updated the task description. (Show Details)

Jdlrobson added a subscriber: Edtadros.

Jdlrobson moved this task from Upcoming to Ready for Development on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.Aug 19 2020, 5:24 PM

Jdlrobson moved this task from Ready for Development to Code Review on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.Aug 19 2020, 7:23 PM

Hi @Jdlrobson, I tested this and the patch but unfortunately could not reproduce the issue locally, and also could not on English Wikipedia. I also tried on real Mobile device, just in case. All the tests are on Chrome desktop/mobile Browser 84.

Can you try and see if you can reproduce on English Wikipedia https://en.m.wikipedia.org/wiki/User:Ammarpad/sandbox?

On Betawiki, I can reproduce it, and I noted that the HTMLs generated by my local wiki and English Wikipedia match, and subsequently matches the HTML you gave above as "before"

<sup id="cite_ref-2" class="reference"><a href="#cite_note-2">[note 1]</a></sup>

In other words, the HTML generated where it works is different from the one generated on Betawiki (https://en.m.wikipedia.beta.wmflabs.org/wiki/Nested_references) where it does not. It sounds to me like something is interfering with the generated HTML on Beta cluster but not in production. I am not sure what's that yet. (I just want make sure that it's not only me who cannot reproduce on English Wikipedia before checking further what might cause this)

My guess is this relates to Parsoid since the HTML output is different. Maybe someone from Parsing-Team--ARCHIVED ( @cscott ?) can explain the HTML change to references we are seeing on beta cluster?

Jdrewniak claimed this task.Sep 2 2020, 5:11 PM

Per Jdlrobson's findings, this may have to do with the local modification to https://en.wikipedia.beta.wmflabs.org/w/index.php?title=MediaWiki:Cite_reference_link&action=history.

default = [[#$2|[$3]]]
modified = [[#$2|<nowiki>[</nowiki>$3<nowiki>]</nowiki>]]

The on wiki change is, however, not new as it's over a decade old, so probably there has to be something that actually triggered the test failures/regression since they were working before then.

Change 616891 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Harden nested references click handler against different markup

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

ReleaseTaggerBot edited projects, added MW-1.36-notes (1.36.0-wmf.8; 2020-09-08); removed MW-1.36-notes (1.36.0-wmf.2; 2020-07-28).Sep 2 2020, 10:00 PM

Jdlrobson reassigned this task from Jdrewniak to Edtadros.Sep 2 2020, 10:54 PM

Jdlrobson moved this task from Code Review to QA on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.

Jdlrobson added a subscriber: Jdrewniak.

Change 616899 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] QA: Restore nested reference test

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

Maintenance_bot removed a project: Patch-For-Review.Sep 3 2020, 12:10 AM

Mhurd claimed this task.Sep 3 2020, 7:35 PM

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Catalina
Browser: Chrome
Device: MBP
Emulated Device: iPhone X
Test Artifact(s):

QA steps

Visit https://en.m.wikipedia.beta.wmflabs.org/wiki/Nested_references
Click [note 1]
click [1]
Repeat.

nested_refs.mov.gif (958×476 px, 503 KB)

Mhurd updated the task description. (Show Details)Sep 3 2020, 8:39 PM

Mhurd moved this task from QA to QA in Prod on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Catalina
Browser: Chrome
Device: MBP
Emulated Device: iPhone X
Test Artifact(s):

QA steps

Visit https://en.m.wikipedia.org/wiki/User:Montehurd/Nested_references
Click [note 1]
click [1]
Repeat.

nested enwiki.mov.gif (922×578 px, 873 KB)

Looks like prod is on 1.36.0-wmf.6, and even though ReleaseTaggerBot says 1.36.0-wmf.8 it seems to work fine. In the recording attached to this comment I'm not focusing on clicking precisely on the brackets, but I did so on both Safari and Chrome after recording and everything still seemed to work fine... 🤔

Mhurd updated the task description. (Show Details)Sep 3 2020, 11:59 PM

@Jdlrobson Hope I didn't goof things here... I was going on the description when I recorded and only noticed the ticket title mentions the brackets themselves after I had recorded... 🤞

@Mhurd per T258096#6430908 this related to a bug in beta cluster which was thankfully never live on production. We may want to verify however that these changes don't break production in any way when it goes live next Thursday (although that would be very very surprising!).

Test Result - Prod (confirmed `1.36.0-wmf.8`)

Status: ✅ PASS
Environment: enwiki
OS: macOS Catalina
Browser: Chrome
Device: MBP
Emulated Device: iPhone X
Test Artifact(s):

QA steps

Visit https://en.m.wikipedia.org/wiki/User:Montehurd/Nested_references
Click [note 1]
click [1]
Repeat.

nested ref retest.mov.gif (918×578 px, 429 KB)

Mhurd reassigned this task from Mhurd to ovasileva.Sep 10 2020, 10:25 PM

Mhurd updated the task description. (Show Details)

Mhurd moved this task from QA in Prod to Ready for Signoff on the Web-Team-Backlog (Kanbanana-FY-2020-21) board.

Mhurd subscribed.

ovasileva closed this task as Resolved.Sep 11 2020, 8:56 AM

	F31936380: Screen Shot 2020-07-15 at 12.12.21 PM.png
	Jul 15 2020, 7:12 PM

	F31936378: Screen Shot 2020-07-15 at 12.11.52 PM.png
	Jul 15 2020, 7:12 PM

	F32243865: nested_refs.mov.gif
	Sep 3 2020, 8:24 PM

Regression: Nested references do not open if user clicks on [ or ] (which are wrapped in span)Closed, ResolvedPublicActions

Description

Developer notes:

Acceptance criteria

QA Results - Beta

QA Results - Prod

QA Results - Prod (Sept 10, confirmed 1.36.0-wmf.8)

Details

Related Objects

Event Timeline

Test Result - Beta

QA steps

Test Result - Prod

QA steps

Test Result - Prod (confirmed 1.36.0-wmf.8)

QA steps

Regression: Nested references do not open if user clicks on [ or ] (which are wrapped in span)
Closed, ResolvedPublic
Actions

QA Results - Prod (Sept 10, confirmed `1.36.0-wmf.8`)

Test Result - Prod (confirmed `1.36.0-wmf.8`)