Skip to content

[cssom-view] screenX/Y are being redefined as double, should movementX/Y as well? #1634

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

Open
scheib opened this issue Jul 20, 2017 · 3 comments

Comments

@scheib
Copy link

scheib commented Jul 20, 2017

CSSOM View Module ED extends MouseEvent's screenX to a double.

Pointer Lock REC extended MouseEvent by adding readonly attribute long movementX.

Should CSSOM View Module also extend movementX to double? Or, should this be updated only in Pointer Lock 2? (This was initially raised in w3c/pointerlock#24)

@NavidZ
Copy link
Member

NavidZ commented Jan 15, 2018

@EiraGe @dtapuska FYI
Any path for this issue? We are eager to modify this in Chrome as there are issues concerning this matter. I suggest modifying it in PointerLock spec if there are no objections. @scheib are you okay with this?

@scheib
Copy link
Author

scheib commented Jan 16, 2018

CSSOM View Module should include movementX/Y. If a patch is refused, then fall back to PointerLock.

Why? Specifications that amend and change behavior of other specifications adds complexity to the web platform. Ideally upstream specifications are updated to be current whenever possible. Making a change if needed in CSSOM View Module is unfortunate, but doing it twice in CSSOM & in Pointer Lock is even worse.

@NavidZ
Copy link
Member

NavidZ commented Jan 17, 2018

CSSOM View Module should include movementX/Y. If a patch is refused, then fall back to PointerLock.

Does it currently include the integer version of movementX/Ys? I wanted to make sure since I haven't found any.

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

No branches or pull requests

4 participants