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 event order related to insertFromPaste, fixes #117 #123

Merged
merged 3 commits into from
Nov 12, 2021

Conversation

johanneswilm
Copy link
Contributor

@johanneswilm johanneswilm commented Sep 10, 2021

@johanneswilm
Copy link
Contributor Author

Hey @saschanaz you tend to be better than me concerning the correct lingo so I have included as a reviewer.

@travisleithead The issue here is that as far as I understand, the decision was that we wanted to say that there should be a paste event whenever we have a insertFromPaste beforeinput event. But we wanted to add this as a non-normative note so that it is just noted as a hint as we are not sure where the entire event order should be defined long-term.

index.html Outdated Show resolved Hide resolved
index.html Outdated Show resolved Hide resolved
index.html Outdated
</p>
When an <code>"insertFromPaste"</code> <code>beforeinput</code>
event is dispatched, it should be preceded by a
<a href="https://www.w3.org/TR/clipboard-apis/#clipboard-event-paste">paste</a>
Copy link
Member

Choose a reason for hiding this comment

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

Probably need to fix clipboard api spec to export paste definition.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@travisleithead @garykac Any change we can get the paste definition from the clipboard apis exported?

Co-authored-by: Kagami Sascha Rosylight <[email protected]>
@johanneswilm johanneswilm added the Agenda+ Queue this item for discussion at the next WG meeting label Sep 24, 2021
@johanneswilm
Copy link
Contributor Author

Objective with PR has changed: https://www.w3.org/2021/09/24-editing-minutes.html#a03

I have checked all browsers [1] and the paste event is triggered before the beforeinput event on all four browsers.

[1] https://github.com/johanneswilm/input-events-testing

@johanneswilm johanneswilm changed the title Add note about paste event related to insertFromPaste, fixes #117 Add event order related to insertFromPaste, fixes #117 Oct 8, 2021
@css-meeting-bot
Copy link
Member

The Web Editing Working Group just discussed do we have a date to discuss #122?.

The full IRC log of that discussion <Travis> Topic: do we have a date to discuss #122?
<Travis> github: https://github.com//pull/122
<Travis> BoCupp: I do have a demo to look at this...
<Travis> .. I can paste a link that is a pre-read
<Travis> johanneswilm: I think we need a demo to show the proposal, and to see the problems (as implemented in Safari) and then have a span of time to review the details and form an opinion.
<Travis> .. can we get the demos within the next few weeks before the next meeting?
<Travis> BoCupp: I don't think we need to wait a month.
<Travis> johanneswilm: So we can get a demo shortly?
<Travis> action: BoCupp to provide demo showing potential alternatives and problems for the composition commands
<Travis> github: https://github.com//pull/123

Copy link
Member

@travisleithead travisleithead left a comment

Choose a reason for hiding this comment

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

Awesome. Thanks for doing this.

@johanneswilm
Copy link
Contributor Author

@travisleithead should we be migrating from travis to github actions? Or is there another policy in place?

@johanneswilm johanneswilm merged commit 80d8edd into gh-pages Nov 12, 2021
@johanneswilm johanneswilm deleted the 117/note-on-paste-event branch November 12, 2021 16:05
@siusin siusin mentioned this pull request Nov 21, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agenda+ Queue this item for discussion at the next WG meeting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants