PointerEvent: set pressure to 0 unless pointer is down #313

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
@appsforartists
Contributor

appsforartists commented Sep 21, 2016

Nonsensically, pointerup events currently have a non-zero pressure on iOS. As outlined in w3c/pointerevents#146, the spec should be updated to make it clear that if a pointer is up, pressure must be 0. (This is already the case for pointers that don't report granular pressure - this proposal only makes that rule consistent.) This PR also makes PEP's behavior consistent with Firefox on all platforms.

https://github.com/w3c/web-platform-tests also needs to be updated, both to assert that pressure is 0 in pointerup and to test for custom pressure in a pointermove; the current test checks in pointerover.

@jquerybot

This comment has been minimized.

Show comment
Hide comment
@jquerybot

jquerybot Sep 21, 2016

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

@jquerybot jquerybot added CLA: Error and removed CLA: Error labels Sep 21, 2016

@scottgonzalez

This comment has been minimized.

Show comment
Hide comment
@scottgonzalez

scottgonzalez Sep 22, 2016

Member

Google, Inc. is not an author.

Member

scottgonzalez commented Sep 22, 2016

Google, Inc. is not an author.

@appsforartists

This comment has been minimized.

Show comment
Hide comment
@appsforartists

appsforartists Sep 22, 2016

Contributor

@willnorris - thoughts on AUTHORS? Apparently, when Polymer maintained this, they didn't list Google either.

Contributor

appsforartists commented Sep 22, 2016

@willnorris - thoughts on AUTHORS? Apparently, when Polymer maintained this, they didn't list Google either.

scottgonzalez added a commit that referenced this pull request Sep 27, 2016

@scottgonzalez scottgonzalez added this to the 0.4.2 milestone May 7, 2017

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