Skip to content

Conversation

@sopranopillow
Copy link
Contributor

Current Behavior

image

New Behavior

image

Related Issue(s)

#23549

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jun 21, 2022

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit f3e9415:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam
Copy link
Collaborator

fabricteam commented Jun 22, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-avatar
Avatar
46.757 kB
13.737 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
187.299 kB
51.651 kB
react-components
react-components: FluentProvider & webLightTheme
31.883 kB
10.516 kB
🤖 This report was generated against 24ee79da3c6501507c1ab3a8feab0e5f173b0e8b

@fabricteam
Copy link
Collaborator

fabricteam commented Jun 22, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1663 1705 5000
Button mount 1170 1202 5000
FluentProvider mount 2250 2175 5000
FluentProviderWithTheme mount 882 843 10
FluentProviderWithTheme virtual-rerender 803 820 10
FluentProviderWithTheme virtual-rerender-with-unmount 865 917 10
MakeStyles mount 2430 2424 50000

@size-auditor
Copy link

size-auditor bot commented Jun 22, 2022

Asset size changes

⚠️ Insufficient baseline data to detect size changes

Unable to find bundle size details for Baseline commit: 52f6f9d

Possible causes

  • The baseline build 52f6f9d is broken
  • The Size Auditor run for the baseline build 52f6f9d was not triggered

Recommendations

  • Please merge your branch for this Pull request with the latest master build and commit your changes once again

@sopranopillow
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@sopranopillow sopranopillow merged commit e81acf1 into microsoft:master Jun 23, 2022
@sopranopillow sopranopillow deleted the avatargroup-hcm branch June 23, 2022 20:50
rohitpagariya pushed a commit to rohitpagariya/fluentui that referenced this pull request Jun 28, 2022
…CM (microsoft#23636)

* fix:Make AvatarGroup's pie layout dividers white for HCM.

* change files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants