Skip to content

Initial 'source' defn #405

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Initial 'source' defn #405

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions sections/event-wheelevent.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,6 +163,21 @@
The <a>un-initialized value</a> of this attribute MUST be
<code>0</code>.
</dd>

<dt><dfn attribute>source</dfn></dt>
Copy link
Contributor

@flackr flackr Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this attribute needs to be added to the WheelEvent interface above

<dd>
The <code>source</code> attribute is a ready only attribute which
Copy link
Contributor

@flackr flackr Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

s/ready/read

indicates the reason this event has been generated and takes one of
three possible values:
Copy link
Contributor

@flackr flackr Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What value would we expect if you create a WheelEvent? I think this is intended to be specified as the "un-initialized" value.


* 'user': this event has been generated as a result of direct user
action.
* 'momentum': this event has been generated as a result of momentum
scrolling which is an animation that is done after the user stops
directly interacting with the input device.
* 'unknown': the browser could not determine the reason this event
has been generated
Copy link
Contributor

@flackr flackr Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Outside of programmatically generated events, do we know if there are any platforms where we won't be able to determine this? I wonder if we need an unknown value at all.

Copy link
Member Author

@vmpstr vmpstr Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good question. It'd be nice to reserve this as a possible values in case there are uncommon platforms that don't expose this. Also, to your question of uninitialized value, do you think we should add another value or use "unknown" as the default value?

</dd>
</dl>

<h5 id="idl-wheeleventinit">WheelEventInit</h5>
Expand Down Expand Up @@ -265,6 +280,8 @@
| | value of movement of a wheel around the z-axis</li> |
| | <li>{{WheelEvent}}.{{WheelEvent/deltaMode}} : unit indicator (pixels, lines, or |
| | pages) for the deltaX, deltaY, and deltaZ attributes</li> |
| | <li>{{WheelEvent}}.{{WheelEvent/source}} : source of this event (e.g. user or |
| | momentum).</li> |
| | </ul> |
++------------------+--------------------------------------------------------------------------------------+

Expand Down
Loading