Page MenuHomePhabricator

TypeError: Cannot read property 'top' of undefined / TypeError: $revisionWrapper.offset() is undefined
Closed, ResolvedPublic1 Estimated Story Points

Description

144 errors in last 12hrs.
Example: https://logstash.wikimedia.org/app/dashboards#/doc/logstash-*/logstash-2021.05.05?id=WDmvPnkBfVMx58vqqzUS
Board: https://logstash.wikimedia.org/goto/ed115437c17d13db4713c7ec10d0c95b

at setRevisionHovered URL1:20:9
at showTooltipsOnMouseMoveHandler URL1:46:1001
at URL1:44:648
at dispatch URL2:70:219

URL1: https://ja.wikipedia.org/w/load.php?lang=ja&modules=ext.RevisionSlider.HelpDialog%2CPointer%2CRevisionList%2CSlider%2CdialogImages%2Cinit%7Cjquery.ui%7Coojs-ui.styles.icons-moderation&skin=vector&version=16ps2
URL2: https://ja.wikipedia.org/w/load.php?lang=ja&modules=jquery%2Coojs-ui-core%2Coojs-ui-widgets&skin=vector&version=1dspt

Event Timeline

Change 700609 had a related patch set uploaded (by Thiemo Kreuz (WMDE); author: Thiemo Kreuz (WMDE)):

[mediawiki/extensions/RevisionSlider@master] Fix jQuery.offset() calculation possibly failing

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

Change 700609 merged by jenkins-bot:

[mediawiki/extensions/RevisionSlider@master] Fix jQuery.offset() calculation possibly failing

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

Jdlrobson claimed this task.
Jdlrobson added subscribers: WMDE-Fisch, thiemowmde.

No errors in the last 12hrs! yay! Thanks @thiemowmde and @WMDE-Fisch !

thiemowmde set the point value for this task to 1.

There's a new variant of this now (less frequent):

at SliderView.pointerContainerClickHandler  https://ru.wikipedia.org/w/load.php?lang=ru&modules=ext.RevisionSlider.HelpDialog%2CPointer%2CRevisionList%2CSlider%2CdialogImages%2Cinit%7Cjquery.ui%7Cmoment&skin=vector&version=1e4cb:48:303
at HTMLDivElement.<anonymous>  https://ru.wikipedia.org/w/load.php?lang=ru&modules=ext.RevisionSlider.HelpDialog%2CPointer%2CRevisionList%2CSlider%2CdialogImages%2Cinit%7Cjquery.ui%7Cmoment&skin=vector&version=1e4cb:45:276
at HTMLDivElement.dispatch  https://ru.wikipedia.org/w/load.php?lang=ru&modules=ext.RevisionSlider.Settings%2ClazyJs%7Cjquery%2Coojs-ui-core%2Coojs-ui-widgets&skin=vector&version=1nrha:71:214
at HTMLDivElement.elemData.handle  https://ru.wikipedia.org/w/load.php?lang=ru&modules=ext.RevisionSlider.Settings%2ClazyJs%7Cjquery%2Coojs-ui-core%2Coojs-ui-widgets&skin=vector&version=1nrha:67:848

Thanks. I think this it something more serious because it's in a code path that's critical for the functionality of the extension. Are you able to open a new ticket for this?