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

Renaming @fluentui/examples to @fluentui/react-examples #15254

Merged
merged 9 commits into from
Sep 26, 2020

Conversation

khmakoto
Copy link
Member

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ yarn change

Description of changes

This PR renames the @fluentui/examples package to @fluentui/react-examples to be more inline with the rest of the repo. It also updates all imports and references to it as well as the lint-files and lint-staged rules that checked for the examples, docs and snapshots to be moved to react-examples.

Focus areas to test

(optional)

Copy link
Member

@ecraig12345 ecraig12345 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked the diff locally and this looks good, thanks!

@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 37bd2eb:

Sandbox Source
Fluent UI Button Configuration
microsoft/fluentui: codesandbox-react-template Configuration
microsoft/fluentui: codesandbox-react-next-template Configuration
microsoft/fluentui: codesandbox-react-northstar-template Configuration

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 25, 2020

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

Sandbox Source
Fluent UI Button Configuration
microsoft/fluentui: codesandbox-react-template Configuration
microsoft/fluentui: codesandbox-react-next-template Configuration
microsoft/fluentui: codesandbox-react-northstar-template Configuration

@size-auditor
Copy link

size-auditor bot commented Sep 26, 2020

Asset size changes

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

Baseline commit: 369c7a1f7f9a4b6a8379eece7b4812e5e0675ebb (build)

@ecraig12345 ecraig12345 reopened this Sep 26, 2020
@ecraig12345
Copy link
Member

ecraig12345 commented Sep 26, 2020

Really weird storybook issue here... https://screener.io/v2/states/Skgr32fUL.microsoft-fluentui/reactExamples/1024x768/Chrome/activityitem-root-default-0

Trying a new build since I don't think it's related to this change.

image

@ecraig12345 ecraig12345 merged commit a05ade3 into microsoft:master Sep 26, 2020
@khmakoto khmakoto deleted the reactExamples branch September 26, 2020 00:50
@msft-github-bot
Copy link
Contributor

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 836 830 5000
BaseButton mount 935 927 5000
Breadcrumb mount 39964 40213 5000
BreadcrumbNext mount 153964 153171 5000
ButtonNext mount 613 603 5000
Checkbox mount 1632 1667 5000
CheckboxBase mount 1448 1441 5000
CheckboxNext mount 1556 1567 5000
ChoiceGroup mount 5028 5131 5000
ChoiceGroupNext mount 9587 9458 5000
ComboBox mount 923 968 1000
CommandBar mount 7492 7548 1000
ContextualMenu mount 12490 12822 1000
DefaultButton mount 1186 1176 5000
DetailsRow mount 3695 3656 5000
DetailsRowFast mount 3708 3679 5000
DetailsRowNoStyles mount 3478 3524 5000
Dialog mount 1495 1479 1000
DocumentCardTitle mount 1750 1749 1000
Dropdown mount 2731 2657 5000
FocusTrapZone mount 1722 1688 5000
FocusTrapZoneNext mount 1847 1795 5000
FocusZone mount 1764 1807 5000
IconButton mount 1818 1837 5000
Label mount 369 344 5000
Layer mount 1972 1979 5000
LayerNext mount 1922 1911 5000
Link mount 454 458 5000
LinkNext mount 467 487 5000
MenuButton mount 1509 1535 5000
MessageBar mount 2006 2052 5000
MessageBarNext mount 1963 1990 5000
Nav mount 3288 3283 1000
OverflowSet mount 1404 1447 5000
OverflowSetNext mount 1046 1052 5000
Panel mount 1449 1479 1000
Persona mount 891 878 1000
Pivot mount 1453 1493 1000
PivotNext mount 1420 1403 1000
PrimaryButton mount 1307 1300 5000
Rating mount 8077 7800 5000
RatingNext mount 7833 7810 5000
SearchBox mount 1354 1360 5000
SearchBoxNext mount 1378 1379 5000
Shimmer mount 2612 2687 5000
ShimmerNext mount 2579 2652 5000
Slider mount 1517 1550 5000
SliderNext mount 1926 1907 5000
SpinButton mount 5013 5074 5000
SpinButtonNext mount 5132 5153 5000
Spinner mount 396 426 5000
SplitButton mount 3207 3283 5000
Stack mount 536 522 5000
StackWithIntrinsicChildren mount 1832 1871 5000
StackWithTextChildren mount 5054 5053 5000
SwatchColorPicker mount 10543 10559 5000
SwatchColorPickerNext mount 10700 10410 5000
TagPicker mount 2741 2810 5000
TeachingBubble mount 48961 48626 5000
TeachingBubbleNext mount 11356 11459 5000
Text mount 436 431 5000
TextField mount 1418 1389 5000
ThemeProvider mount 3104 3046 5000
ThemeProvider virtual-rerender 593 628 5000
Toggle mount 842 847 5000
ToggleNext mount 818 824 5000
button mount 108 106 5000

Perf Analysis (Fluent)

Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🎯 Avatar.Fluent 0.48 0.48 1:1 2000 950
🦄 Button.Fluent 0.13 0.21 0.62:1 5000 636
🔧 Checkbox.Fluent 0.67 0.37 1.81:1 1000 670
🎯 Dialog.Fluent 0.17 0.23 0.74:1 5000 847
🔧 Dropdown.Fluent 2.88 0.5 5.76:1 1000 2883
🔧 Icon.Fluent 0.16 0.06 2.67:1 5000 808
🎯 Image.Fluent 0.09 0.12 0.75:1 5000 442
🔧 Slider.Fluent 1.52 0.36 4.22:1 1000 1520
🔧 Text.Fluent 0.08 0.03 2.67:1 5000 410
🦄 Tooltip.Fluent 0.12 15.14 0.01:1 5000 585

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AccordionMinimalPerf.default 170 0 Infinity:1
AlertMinimalPerf.default 330 0 Infinity:1
AnimationMinimalPerf.default 443 0 Infinity:1
AttachmentMinimalPerf.default 185 0 Infinity:1
AttachmentSlotsPerf.default 1167 0 Infinity:1
BoxMinimalPerf.default 413 0 Infinity:1
ButtonMinimalPerf.default 199 0 Infinity:1
ButtonSlotsPerf.default 644 0 Infinity:1
ButtonUseCssPerf.default 854 0 Infinity:1
ButtonUseCssNestingPerf.default 1092 0 Infinity:1
CardMinimalPerf.default 644 0 Infinity:1
CarouselMinimalPerf.default 495 0 Infinity:1
ChatDuplicateMessagesPerf.default 447 0 Infinity:1
ChatMinimalPerf.default 709 0 Infinity:1
ChatWithPopoverPerf.default 503 0 Infinity:1
CheckboxMinimalPerf.default 2941 0 Infinity:1
DialogMinimalPerf.default 819 0 Infinity:1
DividerMinimalPerf.default 421 0 Infinity:1
DropdownManyItemsPerf.default 823 0 Infinity:1
DropdownMinimalPerf.default 2922 0 Infinity:1
FlexMinimalPerf.default 342 0 Infinity:1
FormMinimalPerf.default 482 0 Infinity:1
GridMinimalPerf.default 402 0 Infinity:1
HeaderMinimalPerf.default 417 0 Infinity:1
HeaderSlotsPerf.default 841 0 Infinity:1
ImageMinimalPerf.default 441 0 Infinity:1
InputMinimalPerf.default 1329 0 Infinity:1
ItemLayoutMinimalPerf.default 1395 0 Infinity:1
LabelMinimalPerf.default 471 0 Infinity:1
LayoutMinimalPerf.default 458 0 Infinity:1
ListCommonPerf.default 717 0 Infinity:1
ListMinimalPerf.default 564 0 Infinity:1
ListNestedPerf.default 632 0 Infinity:1
ListWith60ListItems.default 974 0 Infinity:1
LoaderMinimalPerf.default 760 0 Infinity:1
MenuMinimalPerf.default 953 0 Infinity:1
MenuButtonMinimalPerf.default 1700 0 Infinity:1
PopupMinimalPerf.default 760 0 Infinity:1
PortalMinimalPerf.default 173 0 Infinity:1
ProviderMergeThemesPerf.default 2009 0 Infinity:1
ProviderMinimalPerf.default 1007 0 Infinity:1
RadioGroupMinimalPerf.default 490 0 Infinity:1
ReactionMinimalPerf.default 470 0 Infinity:1
RefMinimalPerf.default 256 0 Infinity:1
SegmentMinimalPerf.default 429 0 Infinity:1
SkeletonMinimalPerf.default 486 0 Infinity:1
SplitButtonMinimalPerf.default 3947 0 Infinity:1
StatusMinimalPerf.default 817 0 Infinity:1
IconMinimalPerf.default 749 0 Infinity:1
TableManyItemsPerf.default 2432 0 Infinity:1
TableMinimalPerf.default 469 0 Infinity:1
TextMinimalPerf.default 405 0 Infinity:1
CustomToolbarPrototype.default 3838 0 Infinity:1
ToolbarMinimalPerf.default 999 0 Infinity:1
TooltipMinimalPerf.default 849 0 Infinity:1
TreeMinimalPerf.default 964 0 Infinity:1
VideoMinimalPerf.default 693 0 Infinity:1
Avatar.Fluent 950 0 Infinity:1
Button.Fluent 636 0 Infinity:1
Checkbox.Fluent 670 0 Infinity:1
Dialog.Fluent 847 0 Infinity:1
Dropdown.Fluent 2883 0 Infinity:1
Icon.Fluent 808 0 Infinity:1
Image.Fluent 442 0 Infinity:1
Slider.Fluent 1520 0 Infinity:1
Text.Fluent 410 0 Infinity:1
Tooltip.Fluent 585 0 Infinity:1
EmbedMinimalPerf.default 2037 1 2037:1
ButtonOverridesMissPerf.default 1684 1 1684:1
SliderMinimalPerf.default 1592 1 1592:1
TextAreaMinimalPerf.default 565 1 565:1
AvatarMinimalPerf.default 543 1 543:1
TreeWith60ListItems.default 212 1 212:1

@msft-github-bot
Copy link
Contributor

🎉@uifabric/fabric-website@v7.15.18 has been released which incorporates this pull request.:tada:

Handy links:

@msft-github-bot
Copy link
Contributor

🎉@uifabric/fabric-website-resources@v7.8.18 has been released which incorporates this pull request.:tada:

Handy links:

@msft-github-bot
Copy link
Contributor

🎉@uifabric/experiments@v7.33.1 has been released which incorporates this pull request.:tada:

Handy links:

@msft-github-bot
Copy link
Contributor

🎉@fluentui/react-examples@v0.4.0 has been released which incorporates this pull request.:tada:

Handy links:

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