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

Add the missing pointerover/enter events #36

Merged
merged 1 commit into from
Mar 3, 2016

Conversation

NavidZ
Copy link
Member

@NavidZ NavidZ commented Feb 26, 2016

In "Process Pending Pointer Capture" section, likewise the
pointerout/leave that is mentioned in this section in some
cases we need to send pointerover/enter when the capture
is lost and pointer is on another node that is not the
capturing node.

In "Process Pending Pointer Capture" section, likewise the
pointerout/leave that is mentioned in this section in some
cases we need to send pointerover/enter when the capture
is lost and pointer is on another node that is not the
capturing node.
@RByers
Copy link
Contributor

RByers commented Feb 26, 2016

Thanks Navid, this looks reasonable to me! But this space is pretty complicated and I'm not an expert, so let's see if someone from Edge can also review the precise wording before we land it (eg. @teddink?).

Presumably this was also a test hole in the tests, right? Can you submit an update for that as well?

@NavidZ
Copy link
Member Author

NavidZ commented Feb 26, 2016

I haven't looked at the tests for the pointer capturing coverage yet. I will keep this in mind when I'm reviewing and testing those on Chrome to add anything that's missing.

@RByers
Copy link
Contributor

RByers commented Feb 26, 2016

Oh right, of course - we're starting capture implementation now so haven't tried to run those tests yet. I knew that, sorry.

@teddink
Copy link

teddink commented Feb 26, 2016

The change sounds good to me as well, but like Rick, I am not yet an expert in the area so I am reaching out to folks internally like @jacobrossi

@mustaqahmed
Copy link
Member

One quick suggestion while we are into it: it seems to me that the section "Process Pending Pointer Capture" should really belong to the "Pointer Capture" section, perhaps between Sec10.2 & Sec10.3, because it is impossible to follow this section w/o skimming through 10.1 & 10.2 first. Only thing we need in the current section (Sec5.2.1) is a reference to the fact that we need to process pending capture before firing a "non-capture" PointerEvent.

@jacobrossi
Copy link
Member

LGTM and +1 to adding test coverage for this. Regarding moving this section, I'm supportive but let's open a separate issue for that.

RByers added a commit that referenced this pull request Mar 3, 2016
…Capture

Add some missing pointerover/enter events
@RByers RByers merged commit 6d2014b into w3c:gh-pages Mar 3, 2016
@RByers
Copy link
Contributor

RByers commented Mar 3, 2016

Thanks everyone!

@NavidZ NavidZ deleted the fixProcessingPendingPointerCapture branch July 27, 2016 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants