Skip to content

[Bugfix] Update padding on video effects picker#3725

Merged
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/picker-margin-fix
Oct 31, 2023
Merged

[Bugfix] Update padding on video effects picker#3725
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/picker-margin-fix

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

@dmceachernmsft dmceachernmsft commented Oct 30, 2023

What

Add space between the side pane children in the background picker

Why

Fixes spacing
Was:
image
Now:
image

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

How Tested

Validated locally, will update snapshots

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 30, 2023

CallWithChat bundle size is increased❗.

  • Current size: 6402917
  • Base size: 6402887
  • Diff size: 30

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 30, 2023

Chat bundle size is decreased✅.

  • Current size: 1519425
  • Base size: 1519426
  • Diff size: -1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 30, 2023

Calling bundle size is increased❗.

  • Current size: 5595448
  • Base size: 5595418
  • Diff size: 30

@github-actions
Copy link
Copy Markdown
Contributor

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

Lines Statements Functions Branches
Base 20979 / 32676
64.2%
20979 / 32676
64.2%
576 / 994
57.94%
1701 / 2661
63.92%
Current 20979 / 32676
64.2%
20979 / 32676
64.2%
576 / 994
57.94%
1701 / 2661
63.92%
Diff 0 / 0
0%
0 / 0
0%
0 / 0
0%
0 / 0
0%

Signed-off-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 30, 2023

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

Lines Statements Functions Branches
Base 43316 / 67101
64.55%
43316 / 67101
64.55%
888 / 1901
46.71%
2634 / 4145
63.54%
Current 43316 / 67101
64.55%
43316 / 67101
64.55%
888 / 1901
46.71%
2634 / 4145
63.54%
Diff 0 / 0
0%
0 / 0
0%
0 / 0
0%
0 / 0
0%

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

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 Oct 31, 2023
@github-actions github-actions Bot removed the update_snapshots Set this label to request automated update of UI snapshots label Oct 31, 2023
@github-actions
Copy link
Copy Markdown
Contributor

Signed-off-by: Donald McEachern <94866715+dmceachernmsft@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit 85c41a5 into main Oct 31, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/picker-margin-fix branch October 31, 2023 20:03
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.

3 participants