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

Button: Removing unused scss files #15835

Merged
merged 2 commits into from
Nov 4, 2020

Conversation

khmakoto
Copy link
Member

@khmakoto khmakoto commented Nov 4, 2020

Pull request checklist

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

Description of changes

This PR removes the unused scss files in the Button components inside @fluentui/react-button.

Focus areas to test

(optional)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Nov 4, 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 b343759:

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 Nov 4, 2020

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 854 865 5000
BaseButtonCompat mount 930 949 5000
Breadcrumb mount 41609 41944 5000
Checkbox mount 1583 1571 5000
CheckboxBase mount 1324 1287 5000
ChoiceGroup mount 4923 4898 5000
ComboBox mount 964 964 1000
CommandBar mount 9975 10004 1000
ContextualMenu mount 5978 5983 1000
DefaultButtonCompat mount 1178 1159 5000
DetailsRow mount 3700 3708 5000
DetailsRowFast mount 3676 3683 5000
DetailsRowNoStyles mount 3555 3492 5000
Dialog mount 1567 1557 1000
DocumentCardTitle mount 1784 1779 1000
Dropdown mount 3485 3487 5000
FocusTrapZone mount 1825 1819 5000
FocusZone mount 1826 1836 5000
IconButtonCompat mount 2055 1932 5000
Label mount 348 343 5000
Layer mount 1831 1821 5000
Link mount 481 469 5000
MenuButtonCompat mount 1509 1535 5000
MessageBar mount 2046 1999 5000
Nav mount 3415 3331 1000
OverflowSet mount 1053 1057 5000
Panel mount 1526 1507 1000
Persona mount 888 864 1000
Pivot mount 1423 1397 1000
PrimaryButtonCompat mount 1295 1300 5000
Rating mount 7788 7767 5000
SearchBox mount 1358 1351 5000
Shimmer mount 2670 2609 5000
Slider mount 1946 1965 5000
SpinButton mount 5095 5265 5000
Spinner mount 409 415 5000
SplitButtonCompat mount 3315 3269 5000
Stack mount 519 526 5000
StackWithIntrinsicChildren mount 1562 1558 5000
StackWithTextChildren mount 4966 4816 5000
SwatchColorPicker mount 10824 10542 5000
Tabs mount 1379 1434 1000
TagPicker mount 2790 2818 5000
TeachingBubble mount 11625 11607 5000
Text mount 454 433 5000
TextField mount 1460 1418 5000
ThemeProvider mount 2014 1986 5000
ThemeProvider virtual-rerender 656 665 5000
Toggle mount 841 822 5000
button mount 552 579 5000
buttonNative mount 111 115 5000

Perf Analysis (Fluent)

Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🎯 Avatar.Fluent 0.47 0.51 0.92:1 2000 931
🦄 Button.Fluent 0.13 0.25 0.52:1 5000 639
🔧 Checkbox.Fluent 0.68 0.36 1.89:1 1000 683
🎯 Dialog.Fluent 0.17 0.23 0.74:1 5000 845
🔧 Dropdown.Fluent 2.96 0.41 7.22:1 1000 2964
🔧 Icon.Fluent 0.15 0.06 2.5:1 5000 756
🦄 Image.Fluent 0.09 0.13 0.69:1 5000 437
🔧 Slider.Fluent 1.57 0.45 3.49:1 1000 1566
🔧 Text.Fluent 0.08 0.03 2.67:1 5000 411
🦄 Tooltip.Fluent 0.11 0.9 0.12:1 5000 571

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
TreeWith60ListItems.default 229 205 1.12:1
ButtonMinimalPerf.default 212 191 1.11:1
AttachmentMinimalPerf.default 186 169 1.1:1
PopupMinimalPerf.default 768 720 1.07:1
TextMinimalPerf.default 406 381 1.07:1
Image.Fluent 437 410 1.07:1
GridMinimalPerf.default 405 381 1.06:1
RadioGroupMinimalPerf.default 501 478 1.05:1
TreeMinimalPerf.default 982 938 1.05:1
BoxMinimalPerf.default 412 395 1.04:1
ButtonSlotsPerf.default 650 625 1.04:1
PortalMinimalPerf.default 168 162 1.04:1
SegmentMinimalPerf.default 396 381 1.04:1
SkeletonMinimalPerf.default 469 453 1.04:1
ToolbarMinimalPerf.default 1091 1052 1.04:1
Button.Fluent 639 614 1.04:1
AnimationMinimalPerf.default 453 439 1.03:1
CardMinimalPerf.default 618 602 1.03:1
ChatDuplicateMessagesPerf.default 446 431 1.03:1
HeaderMinimalPerf.default 426 412 1.03:1
LayoutMinimalPerf.default 453 439 1.03:1
ListMinimalPerf.default 546 529 1.03:1
ListNestedPerf.default 625 608 1.03:1
LoaderMinimalPerf.default 764 744 1.03:1
MenuMinimalPerf.default 919 896 1.03:1
TableMinimalPerf.default 468 453 1.03:1
TooltipMinimalPerf.default 882 859 1.03:1
Dialog.Fluent 845 818 1.03:1
AttachmentSlotsPerf.default 1210 1181 1.02:1
ButtonUseCssPerf.default 882 865 1.02:1
CarouselMinimalPerf.default 491 483 1.02:1
ChatWithPopoverPerf.default 495 483 1.02:1
DropdownMinimalPerf.default 2988 2927 1.02:1
EmbedMinimalPerf.default 2064 2024 1.02:1
HeaderSlotsPerf.default 871 854 1.02:1
ItemLayoutMinimalPerf.default 1407 1377 1.02:1
LabelMinimalPerf.default 453 443 1.02:1
SliderMinimalPerf.default 1593 1565 1.02:1
SplitButtonMinimalPerf.default 4021 3934 1.02:1
VideoMinimalPerf.default 715 704 1.02:1
CheckboxMinimalPerf.default 2986 2957 1.01:1
DialogMinimalPerf.default 830 824 1.01:1
FlexMinimalPerf.default 324 321 1.01:1
MenuButtonMinimalPerf.default 1687 1666 1.01:1
RefMinimalPerf.default 243 240 1.01:1
TextAreaMinimalPerf.default 557 552 1.01:1
Checkbox.Fluent 683 678 1.01:1
Dropdown.Fluent 2964 2948 1.01:1
Icon.Fluent 756 750 1.01:1
Text.Fluent 411 408 1.01:1
AvatarMinimalPerf.default 514 512 1:1
ChatMinimalPerf.default 661 661 1:1
DropdownManyItemsPerf.default 797 798 1:1
ImageMinimalPerf.default 434 436 1:1
ListWith60ListItems.default 984 987 1:1
Avatar.Fluent 931 933 1:1
Slider.Fluent 1566 1563 1:1
ButtonOverridesMissPerf.default 1740 1764 0.99:1
ButtonUseCssNestingPerf.default 1148 1154 0.99:1
InputMinimalPerf.default 1341 1356 0.99:1
ListCommonPerf.default 728 734 0.99:1
StatusMinimalPerf.default 789 793 0.99:1
TableManyItemsPerf.default 2387 2409 0.99:1
CustomToolbarPrototype.default 3914 3942 0.99:1
Tooltip.Fluent 571 577 0.99:1
AccordionMinimalPerf.default 172 176 0.98:1
FormMinimalPerf.default 492 501 0.98:1
ProviderMergeThemesPerf.default 2015 2046 0.98:1
DividerMinimalPerf.default 411 422 0.97:1
ProviderMinimalPerf.default 996 1022 0.97:1
AlertMinimalPerf.default 321 335 0.96:1
ReactionMinimalPerf.default 432 462 0.94:1
IconMinimalPerf.default 699 741 0.94:1

@khmakoto khmakoto closed this Nov 4, 2020
@khmakoto khmakoto reopened this Nov 4, 2020
@khmakoto
Copy link
Member Author

khmakoto commented Nov 4, 2020

@msft-github-bot merge in 1 minute.

@msft-github-bot
Copy link
Contributor

Hello @khmakoto!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

  • I won't merge this pull request until after the UTC date Wed, 04 Nov 2020 20:55:48 GMT, which is in 1 minute

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".

@size-auditor
Copy link

size-auditor bot commented Nov 4, 2020

Asset size changes

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

Baseline commit: 70498e155281bad717091ac140ba79d9fdcce323 (build)

@msft-github-bot msft-github-bot merged commit 08d28c8 into microsoft:master Nov 4, 2020
@khmakoto khmakoto deleted the buttonSass branch November 4, 2020 22:24
@msft-github-bot
Copy link
Contributor

🎉@fluentui/react-button@v1.0.0-beta.8 has been released which incorporates this pull request.:tada:

Handy links:

SethDonohue pushed a commit to SethDonohue/fluentui that referenced this pull request Nov 23, 2020
#### Pull request checklist

- [ ] Addresses an existing issue: Fixes #0000
- [x] Include a change request file using `$ yarn change`

#### Description of changes

This PR removes the unused scss files in the `Button` components inside `@fluentui/react-button`.

#### Focus areas to test

(optional)
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.

4 participants