Skip to content

[BugFix] CallComposite to control the local tile aspect ratio on mobile#3309

Merged
dmceachernmsft merged 7 commits intomainfrom
dmceachernmsft/localTileRatioByAspect
Jul 4, 2023
Merged

[BugFix] CallComposite to control the local tile aspect ratio on mobile#3309
dmceachernmsft merged 7 commits intomainfrom
dmceachernmsft/localTileRatioByAspect

Conversation

@dmceachernmsft
Copy link
Copy Markdown
Member

What

image
Call composite now controls local tile ratio based on screen aspect ratio

Why

High resolution phones will have their local tile aspect ratio be correct based on their orientation.

How Tested

Locally, snapshots to validate scenarios

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 30, 2023

Chat bundle size is not changed.

  • Current size: 2247819
  • Base size: 2247819
  • Diff size: 0

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 30, 2023

CallWithChat bundle size is increased❗.

  • Current size: 10658691
  • Base size: 10658690
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 30, 2023

Calling bundle size is increased❗.

  • Current size: 10018065
  • Base size: 10018064
  • Diff size: 1

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 4, 2023

@dmceachernmsft dmceachernmsft enabled auto-merge (squash) July 4, 2023 20:19
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 4, 2023

@dmceachernmsft dmceachernmsft merged commit ea1d0fd into main Jul 4, 2023
@dmceachernmsft dmceachernmsft deleted the dmceachernmsft/localTileRatioByAspect branch July 4, 2023 20:40
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