Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Fixing typo in Avatar.types.ts #31054

Merged
merged 2 commits into from Apr 15, 2024
Merged

Conversation

khmakoto
Copy link
Member

Previous Behavior

Avatar component JSDoc had an incorrect spelling of "needed" (was "neeeded").

New Behavior

Avatar component JSDoc has correct spelling of "needed".

image

matt4885 and others added 2 commits April 12, 2024 17:53
Avatar JSDoc comment has mispelled word 'neeeded'. This commit fixes it to 'needed'.
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender-with-unmount 82 69 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 598 627 5000
Button mount 296 296 5000
Field mount 1128 1098 5000
FluentProvider mount 726 712 5000
FluentProviderWithTheme mount 75 78 10
FluentProviderWithTheme virtual-rerender 33 32 10
FluentProviderWithTheme virtual-rerender-with-unmount 82 69 10 Possible regression
MakeStyles mount 844 853 50000
Persona mount 1800 1701 5000
SpinButton mount 1354 1404 5000

Copy link

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.

@fabricteam
Copy link
Collaborator

📊 Bundle size report

✅ No changes found

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@khmakoto khmakoto merged commit ba8deb2 into microsoft:master Apr 15, 2024
19 checks passed
@khmakoto khmakoto deleted the avatarTypo branch April 16, 2024 00:03
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Apr 17, 2024
Co-authored-by: matt4885 <mkempey@gmail.com>
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.

None yet

4 participants