Skip to content

script: Implement Touch::new - #46741

Merged
yezhizhen merged 2 commits into
servo:mainfrom
yezhizhen:touch
Jul 23, 2026
Merged

script: Implement Touch::new#46741
yezhizhen merged 2 commits into
servo:mainfrom
yezhizhen:touch

Conversation

@yezhizhen

Copy link
Copy Markdown
Member

Found this to be missing while doing statistics.

Testing: New passing WPT.

.
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
.
Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
@yezhizhen
yezhizhen requested a review from gterzian as a code owner July 23, 2026 08:34
@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Jul 23, 2026
Comment on lines 5 to 6
[TouchEvent::initTouchEvent]
expected: FAIL

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could we also implement initTouchEvent in this change which is sort of a the twin of this new method?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Cannot find that in spec anymore

@yezhizhen
yezhizhen requested a review from mrobinson July 23, 2026 09:36
@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Jul 23, 2026
@yezhizhen
yezhizhen added this pull request to the merge queue Jul 23, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 23, 2026
Merged via the queue into servo:main with commit ec9c932 Jul 23, 2026
34 checks passed
@yezhizhen
yezhizhen deleted the touch branch July 23, 2026 12:00
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Jul 23, 2026
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.

3 participants