Skip to content

[Click to call] Resize observer fix#3192

Merged
dmceachernmsft merged 7 commits intomainfrom
dmceachernmsft/resizeObserverFix
Jun 13, 2023
Merged

[Click to call] Resize observer fix#3192
dmceachernmsft merged 7 commits intomainfrom
dmceachernmsft/resizeObserverFix

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

Fix issue where no persona size default would trigger a infinite loop in the useLayoutEffect that calculates the persona medallions size.

Why

Prevents the infinite loop runtime error the resize observer would cause

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

How Tested

Ran locally on a CRA application and webpack testing with our package

  • react 16.14.0
  • react 18.2.0

@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) June 13, 2023 16:43
@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2023

CallWithChat bundle size is increased❗.

  • Current size: 10604458
  • Base size: 10604450
  • Diff size: 8

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2023

Chat bundle size is increased❗.

  • Current size: 10241388
  • Base size: 10241380
  • Diff size: 8

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 13, 2023

Calling bundle size is increased❗.

  • Current size: 10184906
  • Base size: 10184898
  • Diff size: 8

@github-actions
Copy link
Copy Markdown
Contributor

@dmceachernmsft dmceachernmsft merged commit c5ecff5 into main Jun 13, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/resizeObserverFix branch June 13, 2023 17:47
JamesBurnside pushed a commit that referenced this pull request Jun 13, 2023
* fix infinite loop

* Change files

* Duplicate change files for beta release

* fix lint
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