W3C home > Mailing lists > Public > public-webrtc-logs@w3.org > February 2022

Re: [mediacapture-region] Why expose produceCropTarget at MediaDevices level? (#11)

From: Elad Alon via GitHub <sysbot+gh@w3.org>
Date: Wed, 02 Feb 2022 11:50:44 +0000
To: public-webrtc-logs@w3.org
Message-ID: <issue_comment.created-1027861250-1643802643-sysbot+gh@w3.org>
Thanks for spinning off.

Would the following be a good summary of the discussion that is remaining in this thread?
* @eladalon1983 prefers exposing MediaDevices.produceCropTarget(Element), citing:
  * Better encapsulation, as CropTarget production is media/RTC-related, as is MediaDevices; Element is very general.
  * In contexts where MediaDevices is missing (eg HTTP), this API would not be exposed, which is preferable.
  * If we agree ([other thread](https://github.com/w3c/mediacapture-region/issues/17)) that we need a Promise, then a method is more natural than an attribute.
* @youennf prefers exposing `Element.cropTarget()`.
  * (I don't want to write "citing X" as I do not feel I can do justice to your position this time. Please help me out here.)

-- 
GitHub Notification of comment by eladalon1983
Please view or discuss this issue at https://github.com/w3c/mediacapture-region/issues/11#issuecomment-1027861250 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config
Received on Wednesday, 2 February 2022 11:50:46 UTC

This archive was generated by hypermail 2.4.0 : Saturday, 6 May 2023 21:19:56 UTC