Bug 1957955 Comment 4 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

There's a confusing difference between synthesizeNativeWheel and synthesizeWheel. The delta direction is opposite direction.

I tried the modified test locally with setting "test.events.async.enabled=true" and with changing the `deltaY` to 50, the test works.(In reply to 
Henrik Skupin [:whimboo][⌚️UTC+2] from comment #2)
> With the changes for DOMWindowUtils and EventUtils (as attached as patch) I tried to update the following test as proposed by Hiro to switch from native events to widget events but I can see different issues with missing events:
> 
> 1) Compared to sending a native event the expected events are not always emitted, especially not when we already are at `scrollTop == 0`

You don't need this change wit the positive `deltaY`.
There's a confusing difference between synthesizeNativeWheel and synthesizeWheel. The delta direction is opposite direction.

I tried the modified test locally with setting "test.events.async.enabled=true" and with changing the `deltaY` to 50, the test works.
(In reply to Henrik Skupin [:whimboo][⌚️UTC+2] from comment #2)
> With the changes for DOMWindowUtils and EventUtils (as attached as patch) I tried to update the following test as proposed by Hiro to switch from native events to widget events but I can see different issues with missing events:
> 
> 1) Compared to sending a native event the expected events are not always emitted, especially not when we already are at `scrollTop == 0`

You don't need this change wit the positive `deltaY`.
There's a confusing difference between synthesizeNativeWheel and synthesizeWheel. The delta direction is opposite direction.

I tried the modified test locally with setting "test.events.async.enabled=true" and with changing the `deltaY` to 50, the test works.

(In reply to Henrik Skupin [:whimboo][⌚️UTC+2] from comment #2)
> With the changes for DOMWindowUtils and EventUtils (as attached as patch) I tried to update the following test as proposed by Hiro to switch from native events to widget events but I can see different issues with missing events:
> 
> 1) Compared to sending a native event the expected events are not always emitted, especially not when we already are at `scrollTop == 0`

You don't need this change wit the positive `deltaY`.
There's a confusing difference between synthesizeNativeWheel and synthesizeWheel. The delta direction is opposite direction.

I tried the modified test locally with setting "test.events.async.enabled=true" and with changing the `deltaY` to 50, the test works.

(In reply to Henrik Skupin [:whimboo][⌚️UTC+2] from comment #2)
> With the changes for DOMWindowUtils and EventUtils (as attached as patch) I tried to update the following test as proposed by Hiro to switch from native events to widget events but I can see different issues with missing events:
> 
> 1) Compared to sending a native event the expected events are not always emitted, especially not when we already are at `scrollTop == 0`

You don't need this change with the positive `deltaY`.

Back to Bug 1957955 Comment 4