1960823 - dvcreservations.com - Unable to scroll

Status: ASSIGNED

(Web Compatibility :: Site Reports, defect, P3)

Description

[railioaie]

Environment:
Operating system: Android 12
Firefox version: Firefox Mobile 138.0

Preconditions:
Clean profile

Steps to reproduce:

1. Navigate to: https://dvcreservations.com/rarefinds.php
2. Scroll down

Expected Behavior:
The page is scrolling down

Actual Behavior:
Unable to scroll

Notes:

• Reproduces regardless of the status of ETP
• Reproduces in firefox-nightly, and firefox-release
• Does not reproduce in chrome

Created from https://github.com/webcompat/web-bugs/issues/151834

Comment 1

[Kagami Rosylight [:saschanaz] (they/them)]

This is a library bug.

The site uses jQuery.nicescroll 3.1.0: https://dvcreservations.com/assets/js/jquery.nicescroll.min.js

https://github.com/inuyaksa/jquery.nicescroll/blob/28d5d2ca376ae81ae8af997e0d913c6054a7515f/jquery.nicescroll.js#L303-L307

//## Firefox 18 nightly build (desktop) false positive (or desktop with touch support)
    if (cap.cantouch&&cap.ismozilla&&!cap.isios) {
      this.istouchcapable = true;
      cap.cantouch = false;  // parse normal desktop events
    }   

And then it enters this branch: https://github.com/inuyaksa/jquery.nicescroll/blob/28d5d2ca376ae81ae8af997e0d913c6054a7515f/jquery.nicescroll.js#L620

... where it sets overflow: hidden to the top level html element.

The intervention would be to pretend it's iOS.

Comment 2

[Thomas Wisniewski [:twisniewski]]

Attachment: Bug 1960823 - add an Android-only JS override for dvcreservations.com to disable niceScroll; r?denschub

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/a26989b0b46b
https://hg.mozilla.org/integration/autoland/rev/aa269e4ad3f5
add an Android-only JS override for dvcreservations.com to disable niceScroll; r=denschub,webcompat-reviewers

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/aa269e4ad3f5

Comment 5

[Thomas Wisniewski [:twisniewski]]

I can scroll on the page now whether or not our intervention is active, so we should be able to remove the intervention.

Comment 6

[Kagami Rosylight [:saschanaz] (they/them)]

Huh, what I didn't catch in comment #1 is c.ismozilla = 'MozAppearance' in e.style;. This is fixed by bug 1977489.

Comment 7

[Raul Bucata]

This seems fixed on the latest Nightly. Leaving this open since there is an intervention in place (works regardless of the status of the intervention).