Re: [ambient-light] Editorial: Drop own powerful feature, require active local video source. (#83)

Some drive-by comments (I'm not a reviewer to either of these specs, but I ran into it at the W3C permissions workshop):

- Usually I read "editorial" as changing wording without changing meaning/behavior. This seems like a big change in behavior. Are we using the word differently in different specs?
- The [use-cases](https://w3c.github.io/ambient-light/#usecases-requirements) in the spec contain some entries that this would not work with - smart home, lighting at the work place... Seems like this is about changing the purpose of the ALS API to be more of an API that supplements video captures (the 3rd and 4th use cases). So I would expect that this change modifies the use cases, and focuses on them in the examples.
- I don't think we should treat the ALS as a 1x1 camera. Because:
   1. It feels wrong (to me, though others at the permissions workshop indicated a similar sentiment).
   1. Do users know where that 1x1 camera is on their phone/laptop? I don't. I can cover or turn my camera away but I have no idea how to physically do that with my ambient light sensor. So if I start a video stream and then physically cover my camera for part of it, the stream would still include ambient light readings, which is probably not what I intended. 
   1. Purpose: cameras measure what you show them, ALS is ambient by definition, meaning it is meant to capture light without user involvement.
   
Suggestion: If the idea here is that you'd get an ambient-light stream together with your video stream (which is kind of how I understand this change, with the only example provided), perhaps a more direct way to do it is that AL would be an additional [MediaStreamTrack](https://developer.mozilla.org/en-US/docs/Web/API/MediaStreamTrack) rather than a bespoke JS API? I think this still has some privacy/permission issues, but at least it would be more purpose-driven than capability-driven.

-- 
GitHub Notification of comment by noamr
Please view or discuss this issue at https://github.com/w3c/ambient-light/pull/83#issuecomment-1356060544 using your GitHub account


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

Received on Saturday, 17 December 2022 06:07:11 UTC