Skip to content

[Bugfix] Update text spacing in the participant item for on hold and calling to make sure it doesn't have a huge gap#3849

Merged
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/onHoldText-update
Dec 5, 2023
Merged

[Bugfix] Update text spacing in the participant item for on hold and calling to make sure it doesn't have a huge gap#3849
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/onHoldText-update

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

reduce space between icon and state text
image

Why

Aligns with designs

https://skype.visualstudio.com/SPOOL/_workitems/edit/3421253

How Tested

inspected locally snapshots to update

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

Calling bundle size is decreased✅.

  • Current size: 5585637
  • Base size: 5585654
  • Diff size: -17

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

Chat bundle size is decreased✅.

  • Current size: 1525118
  • Base size: 1525135
  • Diff size: -17

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

CallWithChat bundle size is decreased✅.

  • Current size: 6398500
  • Base size: 6398517
  • Diff size: -17

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

@azure/communication-react jest test coverage for beta.

Lines Statements Functions Branches
Base 43412 / 68600
63.28%
43412 / 68600
63.28%
903 / 1943
46.47%
2571 / 4147
61.99%
Current 43465 / 68599
63.36%
43465 / 68599
63.36%
903 / 1943
46.47%
2572 / 4153
61.93%
Diff 53 / -1
0.08%
53 / -1
0.08%
0 / 0
0%
1 / 6
-0.06%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

@azure/communication-react jest test coverage for stable.

Lines Statements Functions Branches
Base 21153 / 33043
64.01%
21153 / 33043
64.01%
577 / 999
57.75%
1669 / 2659
62.76%
Current 21142 / 33042
63.98%
21142 / 33042
63.98%
577 / 999
57.75%
1650 / 2646
62.35%
Diff -11 / -1
-0.03%
-11 / -1
-0.03%
0 / 0
0%
-19 / -13
-0.41%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

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.

@dmceachernmsft dmceachernmsft added the update_snapshots Set this label to request automated update of UI snapshots label Dec 1, 2023
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Dec 1, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 1, 2023

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 4, 2023

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 5, 2023

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) December 5, 2023 19:05
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 5, 2023

@dmceachernmsft dmceachernmsft merged commit 1f8574a into main Dec 5, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/onHoldText-update branch December 5, 2023 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants