Skip to content

docs(react-avatar): Adding @internal api access modifier to getInitials#23707

Merged
sopranopillow merged 2 commits into
microsoft:masterfrom
sopranopillow:docs-internal
Jun 23, 2022
Merged

docs(react-avatar): Adding @internal api access modifier to getInitials#23707
sopranopillow merged 2 commits into
microsoft:masterfrom
sopranopillow:docs-internal

Conversation

@sopranopillow

Copy link
Copy Markdown
Contributor

As per #23577 this PR adds @internal to getInitials since it is exported in the package, but not in @fluentui/react-components.

Related Issue(s)

#23577

@codesandbox-ci

Copy link
Copy Markdown

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 d892083:

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

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 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

@size-auditor

size-auditor Bot commented Jun 23, 2022

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: 24ee79da3c6501507c1ab3a8feab0e5f173b0e8b (build)

@fabricteam

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1697 1679 5000
Button mount 1281 1256 5000
FluentProvider mount 2188 2192 5000
FluentProviderWithTheme mount 898 919 10
FluentProviderWithTheme virtual-rerender 894 819 10
FluentProviderWithTheme virtual-rerender-with-unmount 935 887 10
MakeStyles mount 2437 2419 50000

@sopranopillow
sopranopillow merged commit 7a10986 into microsoft:master Jun 23, 2022
@sopranopillow
sopranopillow deleted the docs-internal branch June 23, 2022 19:39
rohitpagariya pushed a commit to rohitpagariya/fluentui that referenced this pull request Jun 28, 2022
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