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

Make the two ways of event creation share a path #603

Merged
merged 3 commits into from
Mar 16, 2018

Conversation

annevk
Copy link
Member

@annevk annevk commented Mar 15, 2018

This introduces the "inner event creation steps" so we don't duplicate a bunch of logic.


Preview | Diff

This introduces the "inner event creation steps" so we don't duplicate a bunch of logic.
@annevk annevk requested a review from domenic March 15, 2018 14:40
@annevk
Copy link
Member Author

annevk commented Mar 15, 2018

The main motivation is the ability to add a specification callback here that can be used for custom initialization and also to point out #600 which is rather bad.

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

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

Good idea. Looks good with some minor issues.

dom.bs Outdated
event is signaling.
<li>
<p>Let <var>event</var> be the result of running the <a>inner event creation steps</a> with
<var>eventInterface</var>, <var>realm</var>, the occurrence that the event is signaling, and
Copy link
Member

Choose a reason for hiding this comment

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

"the time of" the occurence

dom.bs Outdated
timestamp property of corresponding <code>NSEvent</code> objects. So in this case, the
{{Event/timeStamp}} value will be equivalent to the <code>NSEvent</code>'s timestamp offset by
<a>time origin</a>, translated in milliseconds, and with its precision adjusted to meet the
minimum <a>clock resolution</a>.
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure "translated in(to?) milliseconds, and with its precision adjusted to meet the minimum clock resolution" belongs here anymore, since you moved those specifics down into the inner event creation steps and are now just using an abstract concept of "time". Probably these clauses should be moved down too.

Copy link
Member Author

Choose a reason for hiding this comment

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

This in particular is about the occurrence, but I'll see about making it match things a little better.

@annevk annevk merged commit 0283565 into master Mar 16, 2018
@annevk annevk deleted the annevk/event-creation branch March 16, 2018 07:57
dom.bs Outdated
{{Event/timeStamp}} value will be equivalent to the <code>NSEvent</code>'s timestamp offset by
<a>time origin</a>, translated in milliseconds, and with its precision adjusted to meet the minimum
<a>clock resolution</a>.
<p class=note>For example, in macOS the occurrence time for input actions are available via the
Copy link

Choose a reason for hiding this comment

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

...occurrence time for input actions is available via the timestamp property of the corresponding NSEvent object.

(or ...occurrence times for input actions are available via the timestamp property of the corresponding NSEvent objects.)

annevk added a commit that referenced this pull request Mar 17, 2018
@annevk annevk mentioned this pull request Mar 17, 2018
annevk added a commit that referenced this pull request Mar 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants