Page MenuHomePhabricator

[mobile-wmf.1] Scrolling empty Watchlist - TypeError: Cannot read properties of undefined (reading 'top')
Closed, ResolvedPublic1 Estimated Story Points

Description

Note: the Console error is the same as T296863: TypeError: Cannot read properties of undefined (reading 'top')

Steps to reproduce:

  1. On mobile as a user without articles on Watchlist, log in and go to the Watchlist.
  2. Scroll the empty Watchlist - the Console will display the following error:

Screen Shot 2023-03-28 at 6.47.03 PM.png (2,308×1,462 px, 455 KB)

logstash link - 42 hits in last 12 hours

error_class TypeError
error_context.namespace Special
error_context.skin minerva
error_context.special_page EditWatchlist
error_context.version 1.41.0-wmf.1
error_context.wiki enwiki
level ERROR
message TypeError: Cannot read properties of undefined (reading 'top')
meta.dt 2023-03-29T01:50:41.288Z
meta.id 4f8fff7e-6176-41cf-8a46-52d52c1e2a08
meta.request_id 1fe02738-45ad-444f-ba32-ea598c7a6ed9
meta.stream mediawiki.client.error
normalized_message TypeError: Cannot read properties of undefined (reading 'top')
	
stack_trace
at n.scrollNearEnd  <anonymous>:182:229
at n._onScroll  <anonymous>:182:73
at OO.EventEmitter.emit  <anonymous>:279:656
at <anonymous>:126:743
at run  <anonymous>:359:987

QA Results - Beta

ACStatusDetails
1T333417#9325208

QA Results - Prod

ACStatusDetails
1T333417#9343400

Details

Other Assignee
Jdlrobson
Related Changes in Gerrit:

Event Timeline

Jdlrobson raised the priority of this task from Low to High.Oct 27 2023, 5:32 PM

The error rate has been rising to the point this is causing a lot of log spam.

Change 971443 had a related patch set uploaded (by Mabualruz; author: Mabualruz):

[mediawiki/extensions/MobileFrontend@master] Scrolling empty Watchlist

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

Change 971443 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Scrolling empty Watchlist

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

Edtadros subscribed.

Test Result - Beta

Status: ✅ PASS
Environment: beta
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

On mobile as a user without articles on Watchlist, log in and go to the Watchlist.
✅ AC1: Scroll the empty Watchlist - the Console will display the following error:

I tried on both Advanced Mode and non-Advanced, no errors appeared in the console.

screenshot 202.png (1,323×833 px, 109 KB)

screenshot 203.png (1,278×889 px, 116 KB)

Sign off involves verifying that the error rate has disappeared on logstash.

The error message is still firing even just checking for the last 2 hours. I will wait another 24 hours before sign off.

Screenshot 2023-11-16 at 12.01.20 PM.png (2,580×538 px, 94 KB)

Test Result - Prod

Status: ✅ PASS
Environment: enwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA

Test Artifact(s):

QA Steps

On mobile as a user without articles on Watchlist, log in and go to the Watchlist.
✅ AC1: Scroll the empty Watchlist - the Console will display the following error:

I tried on both Advanced Mode and non-Advanced, no errors appeared in the console.

screenshot 267.png (1,221×1,148 px, 112 KB)

screenshot 268.png (1,207×1,033 px, 139 KB)