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

Created shim packages #24780

Merged
merged 5 commits into from
Sep 15, 2022
Merged

Conversation

GeoffCoxMSFT
Copy link
Member

@GeoffCoxMSFT GeoffCoxMSFT commented Sep 13, 2022

Changes

Added packages that will contain shims for button, menu, stack, and theme.
These are the shell packages created by the create-package script

Issues

Updates #24675

@GeoffCoxMSFT GeoffCoxMSFT self-assigned this Sep 13, 2022
@GeoffCoxMSFT GeoffCoxMSFT requested review from micahgodbolt and a team September 13, 2022 20:56
@github-actions github-actions bot added this to the July Project Cycle Q3 2022 milestone Sep 13, 2022
@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 f032663:

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

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1285 1284 5000
Button mount 956 957 5000
FluentProvider mount 1576 1594 5000
FluentProviderWithTheme mount 630 626 10
FluentProviderWithTheme virtual-rerender 586 593 10
FluentProviderWithTheme virtual-rerender-with-unmount 636 625 10
MakeStyles mount 1872 1897 50000
SpinButton mount 2508 2515 5000

@size-auditor
Copy link

size-auditor bot commented Sep 13, 2022

Asset size changes

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

Baseline commit: e0a0ab1481d0fb64f2a9804376ad17e4f4f8ea01 (build)

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
global-context
createContext
533 B
341 B
global-context
createContextSelector
554 B
348 B
priority-overflow
createOverflowManager
2.936 kB
1.212 kB
react-accordion
Accordion (including children components)
78.914 kB
24.06 kB
react-alert
Alert
83.511 kB
20.921 kB
react-avatar
Avatar
48.381 kB
13.696 kB
react-avatar
AvatarGroup
14.95 kB
5.989 kB
react-avatar
AvatarGroupItem
68.349 kB
19.039 kB
react-badge
Badge
22.6 kB
7.205 kB
react-badge
CounterBadge
23.503 kB
7.497 kB
react-badge
PresenceBadge
24.05 kB
7.067 kB
react-button
Button
36.119 kB
9.647 kB
react-button
CompoundButton
43.144 kB
10.86 kB
react-button
MenuButton
38.737 kB
10.521 kB
react-button
SplitButton
46.136 kB
11.9 kB
react-button
ToggleButton
51.888 kB
11.127 kB
react-card
Card - All
67.002 kB
19.261 kB
react-card
Card
62.684 kB
18.177 kB
react-card
CardFooter
8.561 kB
3.601 kB
react-card
CardHeader
9.604 kB
3.94 kB
react-card
CardPreview
8.662 kB
3.656 kB
react-combobox
Combobox (including child components)
74.636 kB
24.186 kB
react-combobox
Dropdown (including child components)
74.236 kB
24.086 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
187.939 kB
52.05 kB
react-components
react-components: FluentProvider & webLightTheme
33.359 kB
11.004 kB
react-dialog
Dialog (including children components)
81.048 kB
24.208 kB
react-divider
Divider
16.459 kB
5.902 kB
react-image
Image
10.78 kB
4.264 kB
react-input
Input
23.598 kB
7.661 kB
react-label
Label
9.338 kB
3.86 kB
react-link
Link
11.784 kB
4.867 kB
react-menu
Menu (including children components)
115.735 kB
35.419 kB
react-menu
Menu (including selectable components)
118.934 kB
35.916 kB
react-overflow
hooks only
10.685 kB
4.104 kB
react-popover
Popover
102.938 kB
31.542 kB
react-portal
Portal
10.576 kB
3.875 kB
react-portal-compat
PortalCompatProvider
5.851 kB
1.964 kB
react-positioning
usePositioning
19.7 kB
7.404 kB
react-provider
FluentProvider
15.755 kB
5.883 kB
react-radio
Radio
35.56 kB
11.929 kB
react-radio
RadioGroup
14.248 kB
5.7 kB
react-select
Select
20.846 kB
7.346 kB
react-slider
Slider
31.526 kB
10.046 kB
react-spinbutton
SpinButton
43.943 kB
12.382 kB
react-spinner
Spinner
19.977 kB
6.438 kB
react-switch
Switch
32.097 kB
10.27 kB
react-text
Text - Default
11.782 kB
4.605 kB
react-text
Text - Wrappers
15.092 kB
5.044 kB
react-textarea
Textarea
23.988 kB
8.011 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
29.548 kB
6.434 kB
react-theme
Teams: Light theme
17.452 kB
5.054 kB
react-tooltip
Tooltip
41.502 kB
14.623 kB
react-utilities
SSRProvider
180 B
159 B
🤖 This report was generated against e0a0ab1481d0fb64f2a9804376ad17e4f4f8ea01

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
TreeWith60ListItems.default 186 164 1.13:1
BoxMinimalPerf.default 374 336 1.11:1
LabelMinimalPerf.default 443 399 1.11:1
ChatWithPopoverPerf.default 431 391 1.1:1
PortalMinimalPerf.default 184 171 1.08:1
TextAreaMinimalPerf.default 560 519 1.08:1
FlexMinimalPerf.default 308 289 1.07:1
RadioGroupMinimalPerf.default 506 475 1.07:1
HeaderMinimalPerf.default 397 376 1.06:1
LayoutMinimalPerf.default 403 381 1.06:1
StatusMinimalPerf.default 760 719 1.06:1
ToolbarMinimalPerf.default 1036 985 1.05:1
AttachmentMinimalPerf.default 167 160 1.04:1
AttachmentSlotsPerf.default 1203 1157 1.04:1
ChatDuplicateMessagesPerf.default 319 308 1.04:1
DividerMinimalPerf.default 400 383 1.04:1
GridMinimalPerf.default 358 344 1.04:1
ListMinimalPerf.default 556 535 1.04:1
FormMinimalPerf.default 430 419 1.03:1
HeaderSlotsPerf.default 841 817 1.03:1
ReactionMinimalPerf.default 417 404 1.03:1
RefMinimalPerf.default 226 220 1.03:1
SliderMinimalPerf.default 1721 1673 1.03:1
TreeMinimalPerf.default 874 849 1.03:1
AnimationMinimalPerf.default 585 572 1.02:1
ButtonOverridesMissPerf.default 1442 1416 1.02:1
EmbedMinimalPerf.default 4055 3976 1.02:1
ListWith60ListItems.default 652 640 1.02:1
MenuButtonMinimalPerf.default 1875 1847 1.02:1
SegmentMinimalPerf.default 379 371 1.02:1
SplitButtonMinimalPerf.default 4839 4757 1.02:1
TableManyItemsPerf.default 2154 2116 1.02:1
ButtonSlotsPerf.default 606 598 1.01:1
DropdownMinimalPerf.default 2823 2791 1.01:1
ListCommonPerf.default 699 694 1.01:1
LoaderMinimalPerf.default 710 701 1.01:1
MenuMinimalPerf.default 916 907 1.01:1
ProviderMinimalPerf.default 413 407 1.01:1
TooltipMinimalPerf.default 2519 2494 1.01:1
CardMinimalPerf.default 608 610 1:1
CarouselMinimalPerf.default 505 503 1:1
ChatMinimalPerf.default 792 790 1:1
ImageMinimalPerf.default 450 448 1:1
ItemLayoutMinimalPerf.default 1302 1300 1:1
TableMinimalPerf.default 438 436 1:1
TextMinimalPerf.default 363 363 1:1
CustomToolbarPrototype.default 2791 2788 1:1
AccordionMinimalPerf.default 161 163 0.99:1
AvatarMinimalPerf.default 211 214 0.99:1
CheckboxMinimalPerf.default 2283 2296 0.99:1
DatepickerMinimalPerf.default 6082 6133 0.99:1
InputMinimalPerf.default 1219 1233 0.99:1
ProviderMergeThemesPerf.default 1283 1290 0.99:1
SkeletonMinimalPerf.default 370 372 0.99:1
IconMinimalPerf.default 737 746 0.99:1
VideoMinimalPerf.default 762 768 0.99:1
ButtonMinimalPerf.default 183 186 0.98:1
DialogMinimalPerf.default 827 841 0.98:1
DropdownManyItemsPerf.default 737 751 0.98:1
ListNestedPerf.default 602 615 0.98:1
RosterPerf.default 2346 2383 0.98:1
PopupMinimalPerf.default 655 668 0.98:1
AlertMinimalPerf.default 282 300 0.94:1

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react)

Scenario Render type Master Ticks PR Ticks Iterations Status
buttonNative mount 100 102 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 743 770 5000
Breadcrumb mount 2306 2305 1000
Checkbox mount 2238 2225 5000
CheckboxBase mount 1952 1920 5000
ChoiceGroup mount 3815 3798 5000
ComboBox mount 742 761 1000
CommandBar mount 8930 8839 1000
ContextualMenu mount 9740 9811 1000
DefaultButton mount 962 979 5000
DetailsRow mount 2982 2956 5000
DetailsRowFast mount 2954 3038 5000
DetailsRowNoStyles mount 2823 2824 5000
Dialog mount 2595 2546 1000
DocumentCardTitle mount 153 157 1000
Dropdown mount 2769 2757 5000
FocusTrapZone mount 1545 1554 5000
FocusZone mount 1499 1514 5000
IconButton mount 1373 1373 5000
Label mount 301 335 5000
Layer mount 3729 3759 5000
Link mount 422 413 5000
MenuButton mount 1190 1184 5000
MessageBar mount 1881 1904 5000
Nav mount 2649 2663 1000
OverflowSet mount 972 993 5000
Panel mount 2078 2068 1000
Persona mount 828 836 1000
Pivot mount 1126 1101 1000
PrimaryButton mount 1056 1063 5000
Rating mount 6565 6506 5000
SearchBox mount 1088 1065 5000
Shimmer mount 2486 2458 5000
Slider mount 1689 1708 5000
SpinButton mount 3850 3872 5000
Spinner mount 410 406 5000
SplitButton mount 2411 2473 5000
Stack mount 473 471 5000
StackWithIntrinsicChildren mount 1833 1834 5000
StackWithTextChildren mount 4500 4525 5000
SwatchColorPicker mount 9077 9043 5000
TagPicker mount 1904 1929 5000
TeachingBubble mount 74884 74175 5000
Text mount 394 399 5000
TextField mount 1134 1131 5000
ThemeProvider mount 1017 1017 5000
ThemeProvider virtual-rerender 701 727 5000
ThemeProvider virtual-rerender-with-unmount 1601 1587 5000
Toggle mount 720 704 5000
buttonNative mount 100 102 5000 Possible regression

@Hotell Hotell self-requested a review September 14, 2022 12:16
@GeoffCoxMSFT GeoffCoxMSFT enabled auto-merge (squash) September 15, 2022 17:46
@GeoffCoxMSFT GeoffCoxMSFT merged commit ba749c4 into microsoft:master Sep 15, 2022
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 19, 2022
* master: (28 commits)
  fix: use trigger prop for aria-haspopup (microsoft#24794)
  chore(react-dialog): scaffold DialogContent component (microsoft#24844)
  chore: Northstar screener should read from screenerStates.json (microsoft#24848)
  applying package updates
  (web components) Standardize focus treatment (microsoft#24771)
  Divider - allow default prop override (microsoft#24840)
  GroupedList: fix virtualization (unstable preview) (microsoft#24460)
  fix: Add explicit children prop to TeachingBubble to support React 18 (microsoft#24823)
  feat: Adds `visible` prop to `TableCellActions` (microsoft#24831)
  [Northstar][Dropdown] Fix styling mutation when merging themes (microsoft#24787)
  fix: export `tableCellActionsClassNames` from unstable (microsoft#24830)
  bugfix(react-dialog): Adds color style to DialogSurface (microsoft#24832)
  applying package updates
  Prevent group toggling from selecting the whole group (microsoft#24822)
  feat(react-textarea): Add shadow variant of filled appearance (microsoft#24512)
  applying package updates
  Adding lib-commonjs top-level entries to exports map (microsoft#24792)
  Created shim packages (microsoft#24780)
  feat(react-menu): replace keydown handlers by useARIAButtonShorthand on MenuItem (microsoft#24738)
  fix: update version mismatches triggered by v9 release (microsoft#24812)
  ...
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.

5 participants