Re: [mediacapture-screen-share] Address nullability of DisplayMediaStreamOptions.controller (#236)

Does this need to go to a formal vote in the WG, or can we converge on a decision in GitHub? The opinions I believe I am reading are:
* @eladalon1983: Either decision is OK, but preference for `CaptureController controller;`.
* @beaufortfrancois: Preference for `CaptureController controller;`.
* @youennf: Preference for `CaptureController controller;`.
  * "keeping it as a simple optional member seems good."
* @annevk: Preference for `CaptureController controller;`.
  * "You generally shouldn't need nullable dictionary members to indicate missing values."
* @jan-ivar: Preference for `CaptureController controller = null;`.

If I am reading everyone correctly, it's Jan-Ivar on one side and the rest on another. Jan-Ivar, do you believe this is an important issue, or can we go with the popular vote in this case?

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


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

Received on Monday, 3 October 2022 16:15:33 UTC