Consider using the browser's scroll-behavior for smooth scrolling

[philc]

It seems like all modern browsers now support smooth scrolling as an argument to window.scrollBy. Can we rely on the browser's smooth scrolling implementation and remove the Vimium "use smooth scrolling" and "step size" options?

It seems reasonable to me that if the user has disabled smooth scrolling in the browser, then Vimium will inherit that preference and not have smooth scrolling. What do you think?

The first step is to POC this and see if the browser's implementation is equivalent to Vimium's current implementation.

Related issues:

• Add option to use scrollBehavior:smooth to fix choppy firefox scroll when resistFingerprinting is enabled #3701
• In Firefox, smooth scrolling is very choppy when "privacy.resistFingerprinting" is set to true in about:config #3641
• J/K scrolling polling rate issue? #2650
• Smooth scrolling doesn't feel smooth #4273
• Feature request: smoother scrolling on some websites #3606

[gdh1995]

The smooth scrolling is not as convenient as I once expected: if I hold on j to scroll longer, then a sequence of element.scrollBy({ y: 60, behavior: "smooth"}) will behave very weird, mainly because browser always interrupts an old scrolling and starts a new one.

[bastianpedersen]

I'm not familiar into the intricacies of how scrolling is implemented in Vimium, so I do apologize if my suggestion is strange. Would it be possible to try to change the custom scroll code to as easy as this and see how it behaves? I would love to do it myself, but looking at the existing code I feel completely lost. The code below is taken from a forked Vimari extension and it works really well.

var scrollTarget = -1;
var scrollTargetResetTimeout = null;

function customScrollBy(x, y) {
    if (scrollTarget == -1)
        scrollTarget = window.pageYOffset + y;
    else
        scrollTarget += y;

    window.scroll({ left: x, top: scrollTarget, behavior: 'smooth' });

    clearTimeout(scrollTargetResetTimeout);
    scrollTargetResetTimeout = setTimeout(() => {
      scrollTarget = -1;
    }, 100)
}