Skip to content
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.

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

Actions pointer additional properties #1417

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

timotiusmargo
Copy link

@timotiusmargo timotiusmargo commented May 2, 2019

actions: add additional pointer properties

Add pointer additional properties such as pressure, contact geometry, tilt, etc. to the WebDriver pointer actions in accordance with the latest W3C PointerEvent interface. This additional properties enables automating/simulating new interactions with pointer devices like touch and pen.


Preview | Diff

@timotiusmargo timotiusmargo force-pushed the actions-pointer-additional-properties branch from ee208e3 to 9fb6731 Compare May 2, 2019 02:41
@andreastt andreastt removed their request for review May 2, 2019 12:46
W3C Pointer Events defined additional pointer properties (pressure,
contact geometry, tilt, etc.) to the PointerEvent interface that
reflects new interactions with pointer devices like touch and pen.
Update the WebDriver actions to support these additional properties.
Create optional properties table
Update pointer actions dispatch sections to take
the additional properties into account
@timotiusmargo timotiusmargo force-pushed the actions-pointer-additional-properties branch from 9fb6731 to f7101d4 Compare May 2, 2019 23:06
@thejohnjansen
Copy link
Contributor

I wanted to pop by and see if @AutomatedTester or @shs96c had a chance to look at this one yet. LGTM.

@AutomatedTester
Copy link
Contributor

@timotiusmargo @thejohnjansen This looks good. Are there tests to go with this?

Copy link
Contributor

@AutomatedTester AutomatedTester left a comment

Choose a reason for hiding this comment

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

Needs tests

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