Add audio to Screego

[aydenbottos]

Yes, the whole idea of Screego is to be as simple as possible, but I think the project could benefit from this. Attached are 2 commits that add simple microphone support. Note that these weren't made by me, but rather @L11r. They are older but have no conflicts against upstream.

[BroderPeters]

Screego tries to do one thing well and that is screen sharing. It may seem easy
to just add another video stream, and it probably is. But the cost of
maintaining it isn't worth for me. The next feature request would probably be
microphone support and with that Screego would be competing with services like
jitsi or big blue button and I see no way that I as a solo developer
can do a better job than these projects.

#66 (comment)
Not sure if the maintainer opinion on that changed recently.

[aydenbottos]

Not sure either. If not, I would be happy to fork and rebase when upstream is updated. A second branch could also be an option.

[savely-krasovsky]

Also notice that I had to change Docker image name in order to build binaries and release with Github Actions CI/CD. It's was not intended to be merged sometime. I am not a frontend developer, so my edits probably bad or kinda, someone with right hands should review it and probably rework.

[aydenbottos]

I can have a look sometime although I am not really the best person for the job either! [image: Mailtrack] <https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality7&> Sender notified by Mailtrack <https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality7&> 08/31/21, 09:41:18 PM

…

On Tue, Aug 31, 2021 at 9:20 PM Savely Krasovsky ***@***.***> wrote: Also notice that I had to change Docker image name in order to build binaries and release with Github Actions CI/CD. It's was not intended to be merged sometime. I am not a frontend developer, so my edits probably bad or kinda, someone with right hands should review it and probably rework. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#102 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHNAJAZ7ZTKRHX6GSNXFTFLT7S3G3ANCNFSM5DDKYU2A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[jmattheis]

In the current state, I cannot merge it. The mute button is always visible, even if there's no audio stream and the user cannot decide not to share audio. Screego was never intended to stream audio, thus, it needs to be opt-in.

[aydenbottos]

When a user starts to present, there is an option to share tab audio. The mute button only controls the local audio and allows a user to mute the audio of the stream they are watching. [image: Mailtrack] <https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality9&> Sender notified by Mailtrack <https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality9&> 09/03/21, 09:41:58 AM

…

On Fri, Sep 3, 2021 at 2:02 AM Jannis Mattheis ***@***.***> wrote: In the current state, I cannot merge it. The mute button is always visible, even if there's no audio stream and the user cannot decide not to share audio. Screego was never intended to stream audio, thus, it needs to be opt-in. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#102 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AHNAJA3FJIZCP64GGNUPT6LT76N2LANCNFSM5DDKYU2A> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[jmattheis]

When a user starts to present, there is an option to share tab audio.

But only the first time, if the user allows it, is most likely saved for future requests.

The following things are required for this PR to be merged. (I'll close it for now, because it is not ready)

• The mute button must be hidden, if the other user doesn't share the video with audio.
• There needs to be a separate button or dropdown. "Start presentation with audio" which is only visible if audio sharing is supported by the browser.
• The renames of the docker image names but be reverted.