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.

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

slack-desktop: allow the use of WebRTCPipeWireCapturer flag, adopt #49298

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

mdkcore0
Copy link
Contributor

@mdkcore0 mdkcore0 commented Mar 14, 2024

Slack seems to disable screen sharing on Wayland, this commit allow the use of WebRTCPipeWireCapturer.
Final user still needs to add the flag on the command line to make use of it:
--enable-features=WebRTCPipeWireCapturer --ozone-platform-hint=auto.

Note: just tested on Wayland, it would be great if someone tested it on x11 too.
Note2: As discussed below with @dataCobra, I'm adopting this package.

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture: x86_64-libc

@mdkcore0
Copy link
Contributor Author

@dataCobra :)

@dataCobra
Copy link
Contributor

Thank you for the contribution.

I'll test it under X11.

@dataCobra
Copy link
Contributor

dataCobra commented Mar 20, 2024

I've successfully tested this PR and everything still works under X11.

Since I'm currently not using the slack-desktop app with Void are you able to adopt the package and be the new maintainer?

Because at the moment I can't guaranty that I can update and test the package in the future.

@mdkcore0
Copy link
Contributor Author

hey, I can adopt, we use it for work, so I can keep it updated. Will update this PR to adopt it.
And many thanks for testing it!

@mdkcore0 mdkcore0 changed the title slack-desktop: allow the use of WebRTCPipeWireCapturer flag slack-desktop: allow the use of WebRTCPipeWireCapturer flag, adopt Mar 20, 2024
Slack seems to disable screen sharing on Wayland [1], so this commit
allow the use of WebRTCPipeWireCapturer.
Note that the user still need to add the flag on the command line to
enable it: --enable-features=WebRTCPipeWireCapturer
--ozone-platform-hint=auto.

[1] https://forums.slackcommunity.com/s/question/0D53a00009BSEGACA5/when-will-slack-support-wayland-screen-sharing-does-anyone-have-workarounds-or-hacks-to-make-it-work
@cinerea0 cinerea0 merged commit 2472d00 into void-linux:master Apr 5, 2024
8 checks passed
@mdkcore0 mdkcore0 deleted the update-slack-desktop branch April 5, 2024 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants