Merge pull request #8186 from prlanzarin/2.2-disable-screensharing-safari

Remove screensharing option from Safari by UA checks (#8142)
This commit is contained in:
Chad Pilkey 2019-10-04 13:20:18 -04:00 committed by GitHub
commit b4fd042e87
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -104,6 +104,7 @@ const isMobileBrowser = (BROWSER_RESULTS ? BROWSER_RESULTS.mobile : false)
|| (BROWSER_RESULTS && BROWSER_RESULTS.os
? BROWSER_RESULTS.os.includes('Android') // mobile flag doesn't always work
: false);
const isSafari = BROWSER_RESULTS.name === 'safari';
const DesktopShare = ({
intl,
@ -152,7 +153,12 @@ const DesktopShare = ({
const vDescr = isVideoBroadcasting
? intlMessages.stopDesktopShareDesc : intlMessages.desktopShareDesc;
return (screenSharingCheck && !isMobileBrowser && amIPresenter
const shouldAllowScreensharing = screenSharingCheck
&& !isMobileBrowser
&& amIPresenter
&& !isSafari;
return (shouldAllowScreensharing
? (
<Button
className={cx(styles.button, isVideoBroadcasting || styles.btn)}