Skip to content

[Bugfix] Fix height of control bar so that always has size#2963

Merged
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/controlbarWidthFix
Apr 21, 2023
Merged

[Bugfix] Fix height of control bar so that always has size#2963
dmceachernmsft merged 8 commits intomainfrom
dmceachernmsft/controlbarWidthFix

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Fixes a min height to the control bar container

Why

image
image
stops the control bar from collapsing when there is no people button since main bar has a absolute position

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

How Tested

Inspected locally with people button disabled

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2023

CallWithChat bundle size is increased❗.

  • Current size: 10053931
  • Base size: 10053899
  • Diff size: 32

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2023

Calling bundle size is increased❗.

  • Current size: 9659907
  • Base size: 9659442
  • Diff size: 465

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 21, 2023

Chat bundle size is not changed.

  • Current size: 9702424
  • Base size: 9702424
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

Failed to pass the composite 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.

@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.

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) April 21, 2023 17:54
@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit 8adcefc into main Apr 21, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/controlbarWidthFix branch April 21, 2023 18:07
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