← 返回首页
Fix #66. Make all touch events composed events by hayatoito · Pull Request #67 · w3c/touch-events · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Resetting focus
This repository was archived by the owner on Jul 9, 2024. It is now read-only.

Fix #66. Make all touch events composed events#67

Merged
RByers merged 1 commit into
w3c:gh-pagesfrom
hayatoito:composed
Jun 29, 2016
Merged

Fix #66. Make all touch events composed events#67
RByers merged 1 commit into
w3c:gh-pagesfrom
hayatoito:composed

Conversation

Copy link
Copy Markdown
Member

hayatoito commented Jun 21, 2016
edited
Loading

Regarding #67, make all touch events composed events.

Copy link
Copy Markdown
Member

as I'm not completely au-fait with what composed events actually are/do, nor if there are any issues with a W3C spec referencing WHATWG (political or otherwise) ... @RByers, thoughts?

Copy link
Copy Markdown
Contributor

RByers commented Jun 21, 2016
edited
Loading

The clearest definition of composed seems to be in the note here:

True if event invokes listeners past a ShadowRoot node that is the root of its target attribute value, and false otherwise.

I think the general approach (of extending this spec and referring to the WHATWG DOM spec) is fine. When / if we ever go to publish a CR spec we may need to replace the reference to a W3C version, but presumably by then the W3C fork will be updated to also define composed.

I have a couple questions / concerns with the overall approach though - I'll follow up on WICG/webcomponents#513 and circle back here.

Copy link
Copy Markdown
Member Author

hayatoito commented Jun 22, 2016
edited
Loading

Thank you for the review.

When / if we ever go to publish a CR spec we may need to replace the reference to a W3C version, but presumably by then the W3C fork will be updated to also define composed.

I understand the concern. Actually, I thought the similar thing in creating the PR because it looks that most (all?) of W3C specification do not have a reference to a WHATWG Standard. It looks they try to avoid to have a reference to WHATWG. I am not 100% sure how we proceed in this case.

@annevk do you have any advice? Should I wait W3C version for incorporating Event.composed from WHATWG DOM?

If having a reference to WHATWG DOM is okay, I will try to make a PR for UIEvents at first here w3c/uievents#89. After that, let me ask a merge of PR for Touch Events and Pointer Events, again.

Copy link
Copy Markdown
Member

annevk commented Jun 22, 2016
edited
Loading

There shouldn't be any issue referencing WHATWG documents. Happens all the time. I don't think anyone is maintaining the DOM fork either (which is annoying, since you have to know that is the case, it would be much easier if they just stopped copy-and-pasting without due process).

Copy link
Copy Markdown
Member Author

hayatoito commented Jun 23, 2016
edited
Loading

@annevk Thank you!

Copy link
Copy Markdown
Member Author

@RByers

PR for UI events has been merged w3c/uievents#91.
Could you review this PR now?

Copy link
Copy Markdown
Contributor

RByers commented Jun 29, 2016

LGTM, thanks!

RByers merged commit ec3b943 into w3c:gh-pages Jun 29, 2016
RByers added a commit that referenced this pull request Jun 29, 2016
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Footer

© 2026 GitHub, Inc.