Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pressing the video mute button doesn't turn off the camera #209

Closed
ara4n opened this issue Feb 8, 2022 · 9 comments · Fixed by matrix-org/matrix-js-sdk#3028
Closed

Pressing the video mute button doesn't turn off the camera #209

ara4n opened this issue Feb 8, 2022 · 9 comments · Fixed by matrix-org/matrix-js-sdk#3028
Assignees
@SimonBrandner
Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience Privacy S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements X-Needs-Investigation

Comments

@ara4n
Copy link
Member

ara4n commented Feb 8, 2022

...which is scary and wastes power.
@SimonBrandner
Copy link
Contributor

SimonBrandner commented Feb 8, 2022 •
edited
Loading

By doesn't turn off camera you mean the light next to your webcam?

@ara4n
Copy link
Member Author

ara4n commented Feb 8, 2022

both the light & the camera device itself :)

@SimonBrandner
Copy link
Contributor

Sorry, what I meant is: do you and the other side still see the video from the camera?

@robertlong robertlong added the X-Cannot-Reproduce Needs reproduction steps label Feb 16, 2022
@robertlong robertlong mentioned this issue Feb 16, 2022
Closed
@ara4n ara4n removed the X-Cannot-Reproduce Needs reproduction steps label May 24, 2022
@ara4n
Copy link
Member Author

ara4n commented May 24, 2022

The other side cannot see the video, but the camera is powered on and the green LED is on, which is scary. I can repro every time i hit the video mute button.

@robintown
Copy link
Member

This is probably also due to https://bugs.chromium.org/p/chromium/issues/detail?id=642785

@SimonBrandner
Copy link
Contributor

This is probably also due to bugs.chromium.org/p/chromium/issues/detail?id=642785

That sounds like it only talks about the light - not the camera actually recording

@dbkr dbkr added T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements S-Minor Impairs non-critical functionality or suitable workarounds exist O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Nov 10, 2022
@kegsay
Copy link

kegsay commented Nov 25, 2022

I don't think it's a browser bug, as Jitsi doesn't do this in the same browser/machine.

@germain-gg
Copy link

I just hit the same issue. It looks like the mic indicator on macOs is also shown when muted.

@robintown
Copy link
Member

Per @kegsay's comment this needs re-investigation

@SimonBrandner SimonBrandner self-assigned this Jan 3, 2023
@SimonBrandner SimonBrandner mentioned this issue Jan 5, 2023
Merged
@github-project-automation github-project-automation bot moved this from Backlog to Closed in Element Call issue triage Jan 6, 2023
su-ex added a commit to SchildiChat/matrix-js-sdk that referenced this issue Jan 19, 2023
@su-ex
Merge tag 'v23.1.0' into sc
e6c127a 
* Remove extensible events v1 field population on legacy events ([\matrix-org#3040](matrix-org#3040)).
* Improve hasUserReadEvent and getUserReadUpTo realibility with threads ([\matrix-org#3031](matrix-org#3031)). Fixes element-hq/element-web#24164.
* Remove video track when muting video ([\matrix-org#3028](matrix-org#3028)). Fixes element-hq/element-call#209.
* Make poll start event type available (PSG-962) ([\matrix-org#3034](matrix-org#3034)).
* Add alt event type matching in Relations model ([\matrix-org#3018](matrix-org#3018)).
* Remove usage of v1 Identity Server API ([\matrix-org#3003](matrix-org#3003)).
* Add `device_id` to `/account/whoami` types ([\matrix-org#3005](matrix-org#3005)).
* Implement MSC3912: Relation-based redactions ([\matrix-org#2954](matrix-org#2954)).
* Introduce a mechanism for using the rust-crypto-sdk ([\matrix-org#2969](matrix-org#2969)).
* Support MSC3391: Account data deletion ([\matrix-org#2967](matrix-org#2967)).
* Fix threaded cache receipt when event holds multiple receipts ([\matrix-org#3026](matrix-org#3026)).
* Fix false key requests after verifying new device ([\matrix-org#3029](matrix-org#3029)). Fixes element-hq/element-web#24167 and element-hq/element-web#23333.
* Avoid triggering decryption errors when decrypting redacted events ([\matrix-org#3004](matrix-org#3004)). Fixes element-hq/element-web#24084.
* bugfix: upload OTKs in sliding sync mode ([\matrix-org#3008](matrix-org#3008)).
* Apply edits discovered from sync after thread is initialised ([\matrix-org#3002](matrix-org#3002)). Fixes element-hq/element-web#23921.
* Sliding sync: Fix issue where no unsubs are sent when switching rooms ([\matrix-org#2991](matrix-org#2991)).
* Threads are missing from the timeline ([\matrix-org#2996](matrix-org#2996)). Fixes element-hq/element-web#24036.
* Close all streams when a call ends ([\matrix-org#2992](matrix-org#2992)). Fixes element-hq/element-call#742.
* Resume to-device message queue after resumed sync ([\matrix-org#2920](matrix-org#2920)). Fixes matrix-org/element-web-rageshakes#17170.
* Fix browser entrypoint ([\matrix-org#3051](matrix-org#3051)). Fixes matrix-org#3013.
* Fix failure to start in firefox private browser ([\matrix-org#3058](matrix-org#3058)). Fixes element-hq/element-web#24216.
* Correctly handle limited sync responses by resetting the thread timeline ([\matrix-org#3056](matrix-org#3056)). Fixes element-hq/element-web#23952.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SimonBrandner SimonBrandner

Labels
O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience Privacy S-Minor Impairs non-critical functionality or suitable workarounds exist T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements X-Needs-Investigation
Projects
No open projects
Element Call issue triage
Status: Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove video track when muting video matrix-org/matrix-js-sdk
7 participants
@germain-gg @dbkr @ara4n @robertlong @kegsay @SimonBrandner @robintown