Skip to content

Commit 3c49184

Browse files
[BugFix] Hold button dismiss on mobile (#2904)
* add onlight dismiss to hold action. * Change files * Duplicate change files for beta release * Update packages/react-composites CallWithChatComposite browser test snapshots --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
1 parent 19d8dca commit 3c49184

5 files changed

Lines changed: 15 additions & 0 deletions

File tree

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"type": "patch",
3+
"comment": "Fix hold button behavior to dismiss drawer after call is placed on hold.",
4+
"packageName": "@azure/communication-react",
5+
"email": "94866715+dmceachernmsft@users.noreply.github.com",
6+
"dependentChangeType": "patch"
7+
}
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"type": "patch",
3+
"comment": "Fix hold button behavior to dismiss drawer after call is placed on hold.",
4+
"packageName": "@azure/communication-react",
5+
"email": "94866715+dmceachernmsft@users.noreply.github.com",
6+
"dependentChangeType": "patch"
7+
}

packages/react-composites/src/composites/common/Drawer/MoreDrawer.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,7 @@ export const MoreDrawer = (props: MoreDrawerProps): JSX.Element => {
209209
text: localeStrings.component.strings.holdButton.tooltipOffContent,
210210
onItemClick: () => {
211211
holdButtonProps.onToggleHold();
212+
onLightDismiss();
212213
},
213214
iconProps: { iconName: 'HoldCallContextualMenuItem', styles: { root: { lineHeight: 0 } } }
214215
});

0 commit comments

Comments
 (0)