Skip to content

feat(v9): migrate badge to ship rolluped only dts#23053

Merged
TristanWatanabe merged 2 commits intomicrosoft:masterfrom
TristanWatanabe:react-badge-rolluped-dts
May 18, 2022
Merged

feat(v9): migrate badge to ship rolluped only dts#23053
TristanWatanabe merged 2 commits intomicrosoft:masterfrom
TristanWatanabe:react-badge-rolluped-dts

Conversation

@TristanWatanabe
Copy link
Member

Applied yarn nx workspace-generator migrate-converged-pkg to ship only rolluped type definitions for:

  • react-badge

Related Issue(s)

Fixes partially #22429

@codesandbox-ci
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.

Latest deployment of this branch, based on commit 79fa52b:

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-avatar
Avatar
45.51 kB
13.25 kB
react-badge
Badge
21.1 kB
6.671 kB
react-badge
CounterBadge
22.015 kB
6.976 kB
react-badge
PresenceBadge
22.346 kB
6.715 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
179.74 kB
50.168 kB
react-components
react-components: FluentProvider & webLightTheme
34.178 kB
11.138 kB
🤖 This report was generated against 8ab3a2ac23ee12d9b5fbd72d3b4593fb3691209b

@size-auditor
Copy link

size-auditor bot commented May 18, 2022

Asset size changes

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

Baseline commit: 2769e4b68c65eb8e06cf15384f51059e819b1fc5 (build)

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
Button mount 686 678 5000 Possible regression
FluentProviderWithTheme mount 360 356 10 Possible regression
FluentProviderWithTheme virtual-rerender 300 283 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 363 377 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1120 1116 5000
Button mount 686 678 5000 Possible regression
FluentProvider mount 2285 2246 5000
FluentProviderWithTheme mount 360 356 10 Possible regression
FluentProviderWithTheme virtual-rerender 300 283 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 363 377 10 Possible regression
MakeStyles mount 1928 1857 50000

@TristanWatanabe TristanWatanabe merged commit 2e5bf6d into microsoft:master May 18, 2022
@TristanWatanabe TristanWatanabe deleted the react-badge-rolluped-dts branch May 18, 2022 14:01
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.

5 participants