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

Rename @uifabric/variants to @fluentui/scheme-utilities #15598

Merged
merged 2 commits into from
Oct 20, 2020

Conversation

xugao
Copy link
Contributor

@xugao xugao commented Oct 19, 2020

Pull request checklist

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

Description of changes

rename @uifabric/variants to @fluentui/scheme-utilities

Related issue: #13384

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.

@dzearing should review too but LGTM.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Oct 19, 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 09b68f6:

Sandbox Source
Fluent UI Button Configuration
codesandbox-react-template Configuration
codesandbox-react-northstar-template Configuration

@msft-github-bot
Copy link
Contributor

msft-github-bot commented Oct 19, 2020

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 813 828 5000
BaseButtonCompat mount 909 912 5000
Breadcrumb mount 164636 163700 5000
Checkbox mount 1519 1506 5000
CheckboxBase mount 1289 1245 5000
ChoiceGroup mount 4772 4681 5000
ComboBox mount 984 970 1000
CommandBar mount 22440 22403 1000
ContextualMenu mount 6143 6248 1000
DefaultButtonCompat mount 1143 1135 5000
DetailsRow mount 3628 3593 5000
DetailsRowFast mount 3695 3683 5000
DetailsRowNoStyles mount 3501 3522 5000
Dialog mount 1515 1561 1000
DocumentCardTitle mount 1913 1840 1000
Dropdown mount 3784 3835 5000
FocusTrapZone mount 1844 1822 5000
FocusZone mount 1816 1886 5000
IconButtonCompat mount 1802 1777 5000
Label mount 341 340 5000
Layer mount 1869 1844 5000
Link mount 469 470 5000
MenuButtonCompat mount 1484 1501 5000
MessageBar mount 2016 2053 5000
Nav mount 3320 3310 1000
OverflowSet mount 1096 1014 5000
Panel mount 1469 1475 1000
Persona mount 868 907 1000
Pivot mount 1407 1411 1000
PrimaryButtonCompat mount 1334 1296 5000
Rating mount 7683 7649 5000
SearchBox mount 1312 1321 5000
Shimmer mount 2553 2568 5000
Slider mount 1932 1952 5000
SpinButton mount 5251 5137 5000
Spinner mount 424 430 5000
SplitButtonCompat mount 3209 3138 5000
Stack mount 490 508 5000
StackWithIntrinsicChildren mount 1457 1500 5000
StackWithTextChildren mount 4720 4678 5000
SwatchColorPicker mount 10295 10360 5000
TagPicker mount 2863 2790 5000
TeachingBubble mount 11682 11728 5000
Text mount 425 452 5000
TextField mount 1402 1362 5000
ThemeProvider mount 1973 2006 5000
ThemeProvider virtual-rerender 670 644 5000
Toggle mount 813 795 5000
button mount 562 532 5000
buttonNative mount 135 107 5000

Perf Analysis (Fluent)

Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🎯 Avatar.Fluent 0.45 0.5 0.9:1 2000 906
🦄 Button.Fluent 0.12 0.24 0.5:1 5000 582
🔧 Checkbox.Fluent 0.64 0.34 1.88:1 1000 639
🎯 Dialog.Fluent 0.16 0.22 0.73:1 5000 823
🔧 Dropdown.Fluent 3 0.44 6.82:1 1000 2995
🔧 Icon.Fluent 0.15 0.06 2.5:1 5000 726
🦄 Image.Fluent 0.08 0.13 0.62:1 5000 391
🔧 Slider.Fluent 1.6 0.43 3.72:1 1000 1602
🔧 Text.Fluent 0.07 0.03 2.33:1 5000 373
🦄 Tooltip.Fluent 0.12 0.92 0.13:1 5000 579

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
ButtonSlotsPerf.default 630 586 1.08:1
CarouselMinimalPerf.default 486 454 1.07:1
ChatWithPopoverPerf.default 500 468 1.07:1
FormMinimalPerf.default 445 417 1.07:1
GridMinimalPerf.default 377 352 1.07:1
VideoMinimalPerf.default 654 616 1.06:1
BoxMinimalPerf.default 384 366 1.05:1
TextAreaMinimalPerf.default 522 498 1.05:1
AnimationMinimalPerf.default 439 422 1.04:1
AvatarMinimalPerf.default 504 485 1.04:1
ChatMinimalPerf.default 647 625 1.04:1
EmbedMinimalPerf.default 2015 1939 1.04:1
LabelMinimalPerf.default 444 425 1.04:1
RadioGroupMinimalPerf.default 469 452 1.04:1
IconMinimalPerf.default 705 677 1.04:1
DividerMinimalPerf.default 396 383 1.03:1
ListNestedPerf.default 604 587 1.03:1
TreeWith60ListItems.default 214 208 1.03:1
Text.Fluent 373 361 1.03:1
AlertMinimalPerf.default 314 309 1.02:1
ButtonMinimalPerf.default 197 193 1.02:1
LayoutMinimalPerf.default 433 425 1.02:1
ListCommonPerf.default 668 652 1.02:1
LoaderMinimalPerf.default 761 746 1.02:1
MenuMinimalPerf.default 885 871 1.02:1
ProviderMergeThemesPerf.default 2175 2134 1.02:1
TextMinimalPerf.default 375 369 1.02:1
CustomToolbarPrototype.default 4009 3939 1.02:1
TooltipMinimalPerf.default 831 814 1.02:1
Button.Fluent 582 571 1.02:1
Dialog.Fluent 823 806 1.02:1
ButtonUseCssNestingPerf.default 1104 1095 1.01:1
DropdownManyItemsPerf.default 769 758 1.01:1
DropdownMinimalPerf.default 3032 3011 1.01:1
FlexMinimalPerf.default 321 319 1.01:1
HeaderMinimalPerf.default 391 388 1.01:1
PopupMinimalPerf.default 727 717 1.01:1
PortalMinimalPerf.default 170 168 1.01:1
ProviderMinimalPerf.default 1015 1007 1.01:1
SplitButtonMinimalPerf.default 3828 3798 1.01:1
ToolbarMinimalPerf.default 971 957 1.01:1
TreeMinimalPerf.default 912 907 1.01:1
Dropdown.Fluent 2995 2958 1.01:1
Icon.Fluent 726 718 1.01:1
Slider.Fluent 1602 1579 1.01:1
Tooltip.Fluent 579 572 1.01:1
AttachmentSlotsPerf.default 1143 1146 1:1
ButtonUseCssPerf.default 840 838 1:1
DialogMinimalPerf.default 807 806 1:1
HeaderSlotsPerf.default 797 801 1:1
InputMinimalPerf.default 1321 1320 1:1
ListMinimalPerf.default 501 499 1:1
ListWith60ListItems.default 939 941 1:1
ReactionMinimalPerf.default 417 417 1:1
SegmentMinimalPerf.default 382 381 1:1
SkeletonMinimalPerf.default 447 446 1:1
Avatar.Fluent 906 906 1:1
AttachmentMinimalPerf.default 163 164 0.99:1
ButtonOverridesMissPerf.default 1713 1737 0.99:1
CardMinimalPerf.default 569 573 0.99:1
MenuButtonMinimalPerf.default 1604 1618 0.99:1
SliderMinimalPerf.default 1595 1608 0.99:1
StatusMinimalPerf.default 726 733 0.99:1
TableManyItemsPerf.default 2160 2171 0.99:1
CheckboxMinimalPerf.default 2871 2922 0.98:1
ImageMinimalPerf.default 402 409 0.98:1
ItemLayoutMinimalPerf.default 1304 1326 0.98:1
TableMinimalPerf.default 432 440 0.98:1
ChatDuplicateMessagesPerf.default 427 438 0.97:1
RefMinimalPerf.default 244 252 0.97:1
Image.Fluent 391 403 0.97:1
Checkbox.Fluent 639 665 0.96:1
AccordionMinimalPerf.default 157 171 0.92:1

@size-auditor
Copy link

size-auditor bot commented Oct 19, 2020

Asset size changes

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

Baseline commit: ddb628724b27b6be3d430ea0a7b728ac9adfb8ec (build)

@xugao xugao requested a review from phkuo October 20, 2020 00:52
@xugao xugao force-pushed the xgao/rename_variants_package branch from f595958 to 09b68f6 Compare October 20, 2020 01:56
@xugao xugao merged commit 7547487 into microsoft:master Oct 20, 2020

The following example uses [getTheme()](https://github.com/microsoft/fluentui/blob/master/packages/style-utilities/src/styles/theme.ts) to grab the current global theme, then creates a strong variant of the that theme, and applies that strong variant to the components wrapped by Customizer.
The following example creates a strong variant of a given theme, and applies that strong variant to the components wrapped by `ThemeProvider`.
Copy link
Contributor

Choose a reason for hiding this comment

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

are we renaming "variants" itself too?

@msft-github-bot
Copy link
Contributor

🎉@fluentui/theme-samples@v8.0.0-beta.0 has been released which incorporates this pull request.:tada:

Handy links:

@msft-github-bot
Copy link
Contributor

🎉@fluentui/scheme-utilities@v8.0.0-beta.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.

6 participants