clipboard-apis icon indicating copy to clipboard operation
clipboard-apis copied to clipboard

Change the ClipboardItem constructor to take `ClipboardItemData`.

Open lgarron opened this issue 5 years ago • 8 comments
trafficstars

Closes #????

The following tasks have been completed:

  • [ ] Modified Web platform tests (link to pull request)

Implementation commitment:

  • [ ] WebKit (https://bugs.webkit.org/show_bug.cgi?id=)
  • [ ] Chromium (https://bugs.chromium.org/p/chromium/issues/detail?id=)
  • [ ] Gecko (https://bugzilla.mozilla.org/show_bug.cgi?id=)

Right now:

  • ClipboardItemDataType is a string or a Blob;
  • ClipboardItemData is a Promise of a ClipboardItemDataType.

There are definitely some compatibility issues in the wild. In particular, Chrome only accepts Blobs, and throws an exception if you pass in a string or a Promise. Is it possible that the constructor is supposed to take ClipboardItemDataType values? If not, should it accept any values that are either ClipboardItemData or ClipboardItemDataType?

FYI: my W3C affiliation is set to GitHub, Inc., but this my participation in this repo should be considered under my Chromium affiliation.


Preview | Diff

lgarron avatar Jun 25 '20 10:06 lgarron

FYI: my W3C affiliation is set to GitHub, Inc., but this my participation in this repo should be considered under my Chromium affiliation. I don't know if there's a way to tell the CI this.

lgarron avatar Jun 25 '20 10:06 lgarron

Hi @lgarron. Thanks for these pull requests, we really appreciate your contribution to the Clipboard API.

Re the IRP issue, it'd be difficult for W3C to use Chromium as a sponsor for pull requests, since Chromium hasn't (and probably can't) make IPR commitments.

We are wondering, as a GitHub employee (and thus a Microsoft employee), if your contribution could be covered by Microsoft participation to the WebApps WG?

cc @travisleithead @dontcallmedom

siusin avatar Jun 30 '20 08:06 siusin

We are wondering, as a GitHub employee (and thus a Microsoft employee), if your contribution could be covered by Microsoft participation to the WebApps WG?

Hmm, I'm not contributing this on behalf of GitHub/Microsoft, though. (GitHub has a pretty generous moonlighting clause.) Could I contribute as an individual?

lgarron avatar Jul 06 '20 01:07 lgarron

Hi @lgarron, the key question for IPR commitment purposes is who has rights to the IP in this contribution? For example, if your employer lets employees keep the rights to work done in their own time, and that were the case for work you contributed here, then unless you made another agreement assigning rights, say to chromium, you would be able to make an individual IPR commitment here, and @siusin can send you the link.

Feel free to get in touch with me if you have questions. ([email protected]) Thanks!

wseltzer avatar Jul 06 '20 15:07 wseltzer

@lgarron I'm looking into this right now. Feel free to ping-me offline for questions in the meantime...

travisleithead avatar Jul 08 '20 18:07 travisleithead

@wseltzer, @siusin, to close the loop on this:

Microsoft has no concerns with @lgarron's individual contribution here--it is approved for him to make this contribution as an individual. @siusin please send @lgarron the relevant link to do so.

@lgarron if you plan on making more regular contributions to this group, please let me know, and I'll get you connected with the folks that help coordinate our activities here, we can get you added under Microsoft's contribution agreement. That will remove these speed bumps from your future contributions. ;)

travisleithead avatar Jul 16 '20 19:07 travisleithead

Thanks all!

@lgarron we can consider #126 as a non-normative contribution but please at least sign the Non-Participant Licensing Commitment for #127 via https://www.w3.org/2004/01/pp-impl/nplc/33993598/127/edit?contributors[]=lgarron&groups[]=114929

siusin avatar Jul 17 '20 12:07 siusin

Blocking on template actions. Editors, could you please make sure bugs are filed on the user agents and that there are corresponding tests. It seems @lgarron has identified some additional compat bugs, so would be great to have tests to help resolve those across UAs.

marcoscaceres avatar Jul 20 '20 01:07 marcoscaceres