Skip to content

Conversation

@sopranopillow
Copy link
Contributor

This PR creates the AvatarGroupItem component that will be used by AvatarGroup.

Related Issue(s)

#22240 and #22736

@sopranopillow sopranopillow requested review from a team as code owners May 26, 2022 23:14
@sopranopillow sopranopillow removed request for a team May 26, 2022 23:23
"$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
"extends": "./api-extractor.json",
"mainEntryPointFilePath": "<projectFolder>/dist/packages/react-components/<unscopedPackageName>/src/index.d.ts"
"mainEntryPointFilePath": "<projectFolder>/dist/types/packages/react-components/<unscopedPackageName>/src/index.d.ts"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Out of curiosity, why is this change needed in this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

create-component added it, it doesn't seem to affect anything though. I can remove it just in case?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really sure what this actually does. But if it was added by create-component, and it doesn't break things, then it should be fine.

@codesandbox-ci
Copy link

codesandbox-ci bot commented May 26, 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 84bd8c2:

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

@fabricteam
Copy link
Collaborator

fabricteam commented May 26, 2022

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
181.141 kB
50.598 kB
181.147 kB
50.599 kB
6 B
1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-avatar
Avatar
45.571 kB
13.318 kB
react-components
react-components: FluentProvider & webLightTheme
34.078 kB
11.098 kB
🤖 This report was generated against 02b03646728acb14672b664dcdffe7bd8bac1087

@size-auditor
Copy link

size-auditor bot commented May 26, 2022

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 02b03646728acb14672b664dcdffe7bd8bac1087 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented May 26, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1092 1107 5000
Button mount 693 689 5000
FluentProvider mount 2384 2328 5000
FluentProviderWithTheme mount 388 373 10
FluentProviderWithTheme virtual-rerender 293 298 10
FluentProviderWithTheme virtual-rerender-with-unmount 398 363 10
MakeStyles mount 1938 1879 50000

@sopranopillow sopranopillow merged commit 53dad4c into microsoft:master May 27, 2022
@sopranopillow sopranopillow deleted the avatargroupitem-scaffold branch May 27, 2022 14:19
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants