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

Should relatedTargetScoped flag be set on mousedown/mouseup events? #433

Closed
rniwa opened this issue Mar 11, 2016 · 4 comments
Closed

Should relatedTargetScoped flag be set on mousedown/mouseup events? #433

rniwa opened this issue Mar 11, 2016 · 4 comments

Comments

@rniwa
Copy link
Collaborator

rniwa commented Mar 11, 2016

Do we really want mousedown and mouseup events to have relatedTargetScoped flag set? The following definition in the current spec seems to indicate any event that uses MouseEvent interface to have this flag set:

A trusted event's relatedTargetScoped flag must be initialized to true if the event has a relatedTarget property

Is this an intentional (that any mousedown/mouseup would not propagate out of a shadow root)?

@hayatoito
Copy link
Contributor

mousedown and mouseup events has a non-null related target?

if the event has a relatedTarget property

I think this should be:

if the event has a non-null relatedTarget property value.

@rniwa
Copy link
Collaborator Author

rniwa commented Mar 11, 2016

Yeah, that makes more sense to me.

@rniwa
Copy link
Collaborator Author

rniwa commented Mar 11, 2016

On my second thought, other event such as mousemove can also have null so I'm not certain if that's right either.

@hayatoito
Copy link
Contributor

If it's null, I think it's okay not to set relatedTargetScoped flag to true. This does not affect the actual event path, anyway. The observable difference is the value of Event.relatedTargetScoped flag, which is a minor issue, I think.

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

2 participants