Skip to content
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "Resolve overlap of tooltip and list of devices on mobile configuration page",
"packageName": "@azure/communication-react",
"email": "edwardlee@microsoft.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
{
"type": "patch",
"comment": "Resolve overlap of tooltip and list of devices on mobile configuration page",
"packageName": "@azure/communication-react",
"email": "edwardlee@microsoft.com",
"dependentChangeType": "patch"
}
Original file line number Diff line number Diff line change
Expand Up @@ -114,13 +114,28 @@ export const LocalPreview = (props: LocalPreviewProps): JSX.Element => {
onToggleMicrophone={onToggleMic}
disabled={!microphonePermissionGranted || !hasMicrophones}
showLabel={true}
// disable tooltip as it obscures list of devices on mobile
strings={
props.mobileView ? { tooltipOnContent: '', tooltipOffContent: '', tooltipDisabledContent: '' } : {}
}
styles={localPreviewButtonStyle}
/>
<CameraButton
data-ui-id="call-composite-local-device-settings-camera-button"
{...cameraButtonProps}
showLabel={true}
disabled={!cameraPermissionGranted || !hasCameras}
// disable tooltip as it obscures list of devices on mobile
Comment thread
JamesBurnside marked this conversation as resolved.
strings={
props.mobileView
? {
tooltipOnContent: '',
tooltipOffContent: '',
tooltipDisabledContent: '',
tooltipVideoLoadingContent: ''
}
: {}
}
styles={localPreviewButtonStyle}
/>
{props.showDevicesButton && (
Expand All @@ -130,6 +145,8 @@ export const LocalPreview = (props: LocalPreviewProps): JSX.Element => {
// disable button whilst all other buttons are disabled
disabled={!microphonePermissionGranted || !cameraPermissionGranted || hasNoDevices}
showLabel={true}
// disable tooltip as it obscures list of devices on mobile
strings={props.mobileView ? { tooltipContent: '' } : {}}
styles={devicesButtonStyles}
/>
)}
Expand Down