Skip to content

Fix side pane content being clipped on a small screen at 200% zoom#3937

Merged
JamesBurnside merged 8 commits intomainfrom
jaburnsi/fix-200-zoom-clipping
Dec 23, 2023
Merged

Fix side pane content being clipped on a small screen at 200% zoom#3937
JamesBurnside merged 8 commits intomainfrom
jaburnsi/fix-200-zoom-clipping

Conversation

@JamesBurnside
Copy link
Copy Markdown
Member

What

Set appropriate minWidths and pull out and document constants

Why

At 200+% zoom and 1024x768 resolution, the side panes were super small and incredibly difficult to interact with

How Tested

Local testing at 200+% zoom levels at a 1024x768 resolution

before after
image image

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

CallWithChat bundle size is increased❗.

  • Current size: 6430973
  • Base size: 6430756
  • Diff size: 217

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

Calling bundle size is increased❗.

  • Current size: 5615237
  • Base size: 5615022
  • Diff size: 215

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

Chat bundle size is increased❗.

  • Current size: 1533202
  • Base size: 1533200
  • Diff size: 2

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

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

Lines Statements Functions Branches
Base 21754 / 34321
63.38%
21754 / 34321
63.38%
594 / 1027
57.83%
1693 / 2716
62.33%
Current 21775 / 34321
63.44%
21775 / 34321
63.44%
594 / 1027
57.83%
1707 / 2718
62.8%
Diff 21 / 0
0.06%
21 / 0
0.06%
0 / 0
0%
14 / 2
0.47%

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 21, 2023

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

Lines Statements Functions Branches
Base 43777 / 70167
62.38%
43777 / 70167
62.38%
910 / 1971
46.16%
2592 / 4200
61.71%
Current 43795 / 70190
62.39%
43795 / 70190
62.39%
910 / 1971
46.16%
2611 / 4213
61.97%
Diff 18 / 23
0.01%
18 / 23
0.01%
0 / 0
0%
19 / 13
0.26%

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@JamesBurnside JamesBurnside merged commit 9dec947 into main Dec 23, 2023
@JamesBurnside JamesBurnside deleted the jaburnsi/fix-200-zoom-clipping branch December 23, 2023 15:24
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