Open Bug 1934842 Opened 1 year ago Updated 5 months ago

www.jins.com - Low scroll speed using the mouse wheel

Categories

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

Firefox 135
Desktop
Windows 10

Tracking

(Webcompat Priority:P2, Webcompat Score:4, firefox133 affected, firefox134 affected, firefox135 affected, firefox142 affected)

REOPENED
Webcompat Priority P2
Webcompat Score 4
Tracking Status
firefox133 --- affected
firefox134 --- affected
firefox135 --- affected
firefox142 --- affected

People

(Reporter: ctanase, Unassigned)

References

(Depends on 1 open bug, )

Details

(Keywords: webcompat:platform-bug, webcompat:site-report, Whiteboard: [webcompat-source:web-bugs])

User Story

platform:windows,mac,linux
impact:annoyance
configuration:general
affects:all
branch:release
diagnosis-team:dom
user-impact-score:100

Attachments

(1 file)

Environment:
Operating system: Windows 11/10
Firefox version: Firefox 133.0/135

Steps to reproduce:

  1. Go to https://www.jins.com/jp/guide/product/
  2. Scroll up and down using the mouse wheel.

Expected Behavior:
The page can be scrolled normally.

Actual Behavior:
The scrolling is slow.

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/144552

OS: Windows 11 → Windows 10
Version: unspecified → Firefox 135

Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.

Severity: -- → S3
User Story: (updated)
Priority: -- → P3
Severity: S3 → S4
User Story: (updated)
Webcompat Priority: --- → P2
Webcompat Score: --- → 5
Webcompat Score: 5 → 4

https://www.jins.com/jp/client_info/JINSJINS/view/userweb/js/common/_ws/jp/common/babel/_assets/lib/jquery.verticalScroller.js has this very Gecko specific code:

/*
Firefox super responsive scroll (c) Keith Clark - MIT Licensed
*/
(function(doc) {

  var root = doc.documentElement,
      scrollbarWidth, scrollEvent;

  // Not ideal, but better than UA sniffing.
  if ("MozAppearance" in root.style) {

    // determine the vertical scrollbar width
    scrollbarWidth = root.clientWidth;
    root.style.overflow = "scroll";
    scrollbarWidth -= root.clientWidth;
    root.style.overflow = "";

    // create a synthetic scroll event
    scrollEvent = doc.createEvent("UIEvent")
    scrollEvent.initEvent("scroll", true, true);

    // event dispatcher
    function scrollHandler() {
      doc.dispatchEvent(scrollEvent)
    }

    // detect mouse events in the document scrollbar track
    doc.addEventListener("mousedown", function(e) {
      if (e.clientX > root.clientWidth - scrollbarWidth) {
        doc.addEventListener("mousemove", scrollHandler, false);
        doc.addEventListener("mouseup", function() {
          doc.removeEventListener("mouseup", arguments.callee, false);
          doc.removeEventListener("mousemove", scrollHandler, false);
        }, false)
      }
    }, false)

    // override mouse wheel behaviour.
    doc.addEventListener("DOMMouseScroll", function(e) {
      // Don't disable hot key behaviours
      if (!e.ctrlKey && !e.shiftKey) {
        root.scrollTop += e.detail * 16;
        scrollHandler.call(this, e);
        e.preventDefault()
      }
    }, false)

  }
})(document);

Two things: -moz-appearance and DOMMouseScroll. Preventing the branch with delete CSS2Properties.prototype.MozAppearance fixes the issue.

Depends on: 1977489
No longer depends on: 1351745

No scroll problem anymore on Nightly with bug 1977489 in. 🎉

Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED

Verified. The issue is still reproducible with the RC build.

Tested with :
Firefox 142.0 - candidate build 1
Windows 10

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached video firefox

I said Nightly - 143. Do we keep the bug until it reaches the stable release?

User Story: (updated)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: