-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
Mike/lstream 654/screenshare audio #119
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is incredible work in such a short amount of time! Kudos!
It did present some issues with the canvas sizing, but nothing that can't be quickly resolved.
19e0aa1
to
f9eacc1
Compare
f9eacc1
to
f849e92
Compare
@@ -2012,12 +2079,33 @@ export const commands = (_project: ScenelessProject) => { | |||
}, | |||
}) | |||
}, | |||
setParticipantAudioOnly( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currently, calling setParticipantAudioOnly
on ScreenShare hides the container and also mutes the audio. Making it visible again does not unmute the audio; it remains muted.
This behavior is consistent across Camera participants as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works fine for me on stage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this was glitch. It works well now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is amazing!
No description provided.