Skip to content

Conversation

@vmpstr
Copy link
Member

@vmpstr vmpstr commented Oct 21, 2025

Closes #??

The following tasks have been completed:

  • Confirmed there are no ReSpec/BikeShed errors or warnings.
  • Modified Web platform tests (link to pull request)

Implementation commitment:


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

Choose a reason for hiding this comment

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

s/ready/read

<code>0</code>.
</dd>

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

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
indicates the reason this event has been generated and takes one of
three possible values:
Copy link
Contributor

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.

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

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

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants