Skip to content

chore: migrate react-table to new file structure#25377

Merged
ling1726 merged 4 commits intomicrosoft:masterfrom
ling1726:chore/migrate-table
Oct 27, 2022
Merged

chore: migrate react-table to new file structure#25377
ling1726 merged 4 commits intomicrosoft:masterfrom
ling1726:chore/migrate-table

Conversation

@ling1726
Copy link
Contributor

@ling1726 ling1726 commented Oct 25, 2022

Runs migrate-converged-pkg on the react-table package and fixes the import paths that were broken by moved files.

Updated stories imports to work with new file structure

@ling1726
Copy link
Contributor Author

There are missing exports, will move this back to draft and fix the missing exports first

@ling1726 ling1726 marked this pull request as draft October 25, 2022 20:56
@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 25, 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 833b151:

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

@size-auditor
Copy link

size-auditor bot commented Oct 25, 2022

Asset size changes

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

Baseline commit: 2a210557ea6c23470d7cf803df0f01a3f5dbc307 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Oct 25, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1616 1646 5000
Button mount 1179 1130 5000
FluentProvider mount 1969 1903 5000
FluentProviderWithTheme mount 723 729 10
FluentProviderWithTheme virtual-rerender 682 671 10
FluentProviderWithTheme virtual-rerender-with-unmount 702 714 10
MakeStyles mount 2331 2278 50000
SpinButton mount 3244 3280 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Oct 25, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
62.94 kB
17.663 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
189.603 kB
52.909 kB
react-components
react-components: FluentProvider & webLightTheme
33.446 kB
11.033 kB
react-portal-compat
PortalCompatProvider
5.857 kB
1.978 kB
🤖 This report was generated against 2a210557ea6c23470d7cf803df0f01a3f5dbc307

@ling1726 ling1726 marked this pull request as ready for review October 27, 2022 14:04
@ling1726 ling1726 merged commit 870f51f into microsoft:master Oct 27, 2022
NotWoods pushed a commit to NotWoods/fluentui that referenced this pull request Nov 18, 2022
* chore: migrate react-table to new file structure

* update stories imports

* changefile
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.

4 participants