Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Make Element Call screensharing work on desktop #9476

Merged
merged 3 commits into from
Oct 20, 2022

Conversation

SimonBrandner
Copy link
Contributor

@SimonBrandner SimonBrandner commented Oct 20, 2022

Requires element-hq/element-call#645
Fixes element-hq/element-web#23382


Here's what your changelog entry will look like:

🐛 Bug Fixes

@SimonBrandner SimonBrandner added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems label Oct 20, 2022
@SimonBrandner SimonBrandner requested a review from a team as a code owner October 20, 2022 17:08
src/models/Call.ts Show resolved Hide resolved
@DoM1niC
Copy link

DoM1niC commented Oct 20, 2022

Doesn't fix the Element Desktop
Enabling screensharing error DOMException: Permission denied

Signed-off-by: Šimon Brandner <[email protected]>
Signed-off-by: Šimon Brandner <[email protected]>
@SimonBrandner
Copy link
Contributor Author

Doesn't fix the Element Desktop Enabling screensharing error DOMException: Permission denied

It does work for me, what version of EC are you using?

Comment on lines -833 to -834
this.messaging!.on(`action:${ElementWidgetActions.TileLayout}`, this.onTileLayout);
this.messaging!.on(`action:${ElementWidgetActions.SpotlightLayout}`, this.onSpotlightLayout);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whoops

@SimonBrandner SimonBrandner merged commit 493ee42 into develop Oct 20, 2022
@SimonBrandner SimonBrandner deleted the SimonBrandner/feat/screenshare-ec branch October 20, 2022 18:41
robintown pushed a commit to robintown/matrix-react-sdk that referenced this pull request Oct 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Screensharing in Element Call-powered calls doesn't work on Desktop
4 participants