Skip to content
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

[Bugfix] long name screenshare mobile fix #5677

Merged
merged 8 commits into from
Feb 28, 2025

Conversation

dmceachernmsft
Copy link
Member

What

Update the styling of the spinner on the screenshare component to handle long names better

Why

Stops the length of the name causing weird rendering issues in the composite and components when someone has a really long name from causing scroll bars

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

How Tested

image
Tested locally

Copy link
Contributor

Copy link
Contributor

github-actions bot commented Feb 28, 2025

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

Lines Statements Functions Branches
Base 28017 / 44682
62.7%
28017 / 44682
62.7%
786 / 1436
54.73%
2367 / 3731
63.44%
Current 28021 / 44693
62.69%
28021 / 44693
62.69%
786 / 1436
54.73%
2340 / 3714
63%
Diff 4 / 11
-0.01%
4 / 11
-0.01%
0 / 0
0%
-27 / -17
-0.44%

Copy link
Contributor

github-actions bot commented Feb 28, 2025

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

Lines Statements Functions Branches
Base 58274 / 94248
61.83%
58274 / 94248
61.83%
1177 / 2693
43.7%
3559 / 5843
60.91%
Current 58267 / 94260
61.81%
58267 / 94260
61.81%
1177 / 2693
43.7%
3506 / 5811
60.33%
Diff -7 / 12
-0.02%
-7 / 12
-0.02%
0 / 0
0%
-53 / -32
-0.58%

Copy link
Contributor

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) February 28, 2025 18:49
Copy link
Contributor

Copy link
Contributor

CallWithChat bundle size is not changed.

  • Current size: 12401112
  • Base size: 12401112
  • Diff size: 0

Copy link
Contributor

Chat bundle size is not changed.

  • Current size: 1777281
  • Base size: 1777281
  • Diff size: 0

Copy link
Contributor

Calling bundle size is not changed.

  • Current size: 12401100
  • Base size: 12401100
  • Diff size: 0

Copy link
Contributor

@dmceachernmsft dmceachernmsft merged commit 8fea7e8 into main Feb 28, 2025
41 checks passed
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/long-name-screenshare-mobile branch February 28, 2025 19:30
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