search

w3c / mediacapture-handle

https://w3c.github.io/mediacapture-handle/
Other
14 stars 10 forks source link

Fix event.captureHandle() when not observable #55

Closed eladalon1983 closed 2 years ago

eladalon1983 commented 2 years ago

Fix #53.

eladalon1983 commented 2 years ago

Thanks for raising this, @beaufortfrancois. Your thoughts about this fix?

beaufortfrancois commented 2 years ago

Non-owner LGTM

Note that having https://github.com/w3c/mediacapture-handle/issues/50 settled may make this PR obsolete. @jan-ivar What are your thoughts on https://github.com/w3c/mediacapture-handle/issues/50#issuecomment-1114964918?

eladalon1983 commented 2 years ago

Note that having #50 settled may make this PR obsolete.

Yes, I've mentioned as much on the issue accompanying this PR. It's not clear to me if we can resolve issue #50 quickly, so in the meantime, I'd like to land this incremental improvement to the spec.

eladalon1983 commented 2 years ago

Rendered irrelevant by other changes. Closing.