Skip to content

Commit e0941cc

Browse files
authored
[MM-62241] Fix screen sharing from popout on new Desktop versions (#948)
* Fix screen sharing from popout on new Desktop versions * Fix linting * Fix leaving call from call post in main desktop view
1 parent be75e98 commit e0941cc

File tree

2 files changed

+3
-13
lines changed

2 files changed

+3
-13
lines changed

webapp/src/components/custom_post_types/post_type/component.tsx

+2-5
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,6 @@ import {
2222
callStartedTimestampFn,
2323
getCallPropsFromPost,
2424
getUserDisplayName,
25-
sendDesktopEvent,
26-
shouldRenderDesktopWidget,
2725
toHuman,
2826
untranslatable,
2927
} from 'src/utils';
@@ -74,9 +72,8 @@ const PostType = ({
7472
// NOTE: this also handles the desktop global widget case since the opener window
7573
// will have the client.
7674
callsClient.disconnect();
77-
} else if (shouldRenderDesktopWidget()) {
78-
// DEPRECATED: legacy Desktop API logic (<= 5.6.0)
79-
sendDesktopEvent('calls-leave-call', {callID: post.channel_id});
75+
} else if (window.desktopAPI?.leaveCall) {
76+
window.desktopAPI.leaveCall();
8077
}
8178
};
8279

webapp/src/components/expanded_view/component.tsx

+1-8
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ import React from 'react';
1616
import {OverlayTrigger, Tooltip} from 'react-bootstrap';
1717
import {IntlShape} from 'react-intl';
1818
import {RouteComponentProps} from 'react-router-dom';
19-
import {compareSemVer} from 'semver-parser';
2019
import {hostMuteOthers, hostRemove} from 'src/actions';
2120
import {Badge} from 'src/components/badge';
2221
import CallDuration from 'src/components/call_widget/call_duration';
@@ -73,7 +72,6 @@ import {
7372
isDMChannel,
7473
sendDesktopEvent,
7574
setCallsGlobalCSSVars,
76-
shouldRenderDesktopWidget,
7775
untranslatable,
7876
} from 'src/utils';
7977
import styled, {createGlobalStyle, css} from 'styled-components';
@@ -493,14 +491,9 @@ export default class ExpandedView extends React.PureComponent<Props, State> {
493491
});
494492
this.props.trackEvent(Telemetry.Event.UnshareScreen, Telemetry.Source.ExpandedView, {initiator: fromShortcut ? 'shortcut' : 'button'});
495493
} else if (!this.props.screenSharingSession) {
496-
if (window.desktop && compareSemVer(window.desktop.version, '5.1.0') >= 0) {
497-
this.props.showScreenSourceModal();
498-
} else if (window.desktopAPI?.openScreenShareModal) {
494+
if (window.desktopAPI?.openScreenShareModal) {
499495
logDebug('desktopAPI.openScreenShareModal');
500496
window.desktopAPI.openScreenShareModal();
501-
} else if (shouldRenderDesktopWidget()) {
502-
// DEPRECATED: legacy Desktop API logic (<= 5.6.0)
503-
sendDesktopEvent('desktop-sources-modal-request');
504497
} else {
505498
const stream = await getScreenStream('', hasExperimentalFlag());
506499
if (window.opener && stream) {

0 commit comments

Comments
 (0)