Re: [mediacapture-region] MediaDevices.produceCropTarget -> CropTarget.fromElement (#50)

> I made this API proposal based on the consensus we achieved that all elements would be supported. Given #51 wants to revisit this, this makes me wonder whether we should pause this PR until we resolve #51.

The current PR would still be an improvement over `mediaDevices.produceCropTarget()` and we should therefore merge it. Also, please note this message I had posted on #51:

> @youennf, you have previously agreed HTMLElement made sense, and https://github.com/w3c/mediacapture-region/issues/13, we even expanded to Element. You made this suggestion five months ago. The API as a whole has been debated for 1.5 years by now. I do not think this qualifies as "rushed". It is the nature of our jobs that decisions must be made without full certainty. Past a certain point, additional debates are less productive than making a decision, even if that decision could end up proving less than perfectly optimal. (To clarify - I think we're making a good decision in https://github.com/w3c/mediacapture-region/pull/50.)

Let's press onwards. Many more productive issues await us.

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


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Friday, 27 May 2022 11:24:50 UTC