-
Notifications
You must be signed in to change notification settings - Fork 60
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add context menu icon for removing participant #4562
Add context menu icon for removing participant #4562
Conversation
Chat bundle size is increased❗.
|
Calling bundle size is increased❗.
|
CallWithChat bundle size is increased❗.
|
Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot. |
@azure/communication-react jest test coverage for stable.
|
@azure/communication-react jest test coverage for beta.
|
fb346ff
to
1396851
Compare
Failed to pass the UI Test. If this PR is for UI change and the error is snapshot mismatch, please add "update_snapshots" label to the PR for updating the snapshot. |
…ttps://github.com/Azure/communication-ui-library into mgamis/add-context-menu-icon-to-remove-participant
@@ -321,6 +321,10 @@ export const ParticipantList = (props: ParticipantListProps): JSX.Element => { | |||
itemProps: { | |||
styles: props.styles?.participantItemStyles?.participantSubMenuItemsStyles | |||
}, | |||
iconProps: { | |||
iconName: 'ContextMenuRemoveParticipant', | |||
styles: { root: { lineHeight: 0 } } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit : I think there is a style that we have set you can import so there isn't a magic 0
What
Add context menu icon for removing participant
Why
https://skype.visualstudio.com/SPOOL/_workitems/edit/3687998
How Tested
Local Calling sample testing
Desktop:
Mobile:
Process & policy checklist
Is this a breaking change?