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

Fixing issue #12592- StackItem does not use its padding token #14554

Merged
merged 2 commits into from
Aug 17, 2020

Conversation

pavanpadavala
Copy link
Contributor

Pull request checklist

Description of changes

StackItem styles has a 'padding' token but it is not used in the code. so adding that to fix and function as designed.

Focus areas to test

(optional)

@msft-github-bot
Copy link
Contributor

msft-github-bot commented Aug 17, 2020

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 993 986 5000
ButtonNext mount 632 624 5000
Checkbox mount 1759 1807 5000
CheckboxBase mount 1487 1492 5000
CheckboxNext mount 1913 1832 5000
ChoiceGroup mount 5512 5542 5000
ComboBox mount 1046 1082 1000
CommandBar mount 8629 8578 1000
ContextualMenu mount 16483 16379 1000
DefaultButton mount 1210 1233 5000
DetailsRow mount 4039 4056 5000
DetailsRowFast mount 3971 4082 5000
DetailsRowNoStyles mount 3784 3856 5000
Dialog mount 1738 1714 1000
DocumentCardTitle mount 2028 2030 1000
Dropdown mount 3002 3237 5000
FocusZone mount 1953 2011 5000
IconButton mount 1956 1948 5000
Label mount 348 357 5000
Link mount 500 506 5000
LinkNext mount 519 522 5000
MenuButton mount 1626 1681 5000
Nav mount 3612 3652 1000
Panel mount 1670 1697 1000
Persona mount 963 928 1000
Pivot mount 1573 1647 1000
PivotNext mount 1614 1565 1000
PrimaryButton mount 1405 1391 5000
SearchBox mount 1442 1470 5000
SearchBoxNext mount 1535 1512 5000
Slider mount 1636 1661 5000
SliderNext mount 2104 2100 5000
SpinButton mount 5695 5693 5000
SpinButtonNext mount 5844 5764 5000
Spinner mount 459 446 5000
SplitButton mount 3580 3515 5000
Stack mount 575 565 5000
StackWithIntrinsicChildren mount 2211 2259 5000
StackWithTextChildren mount 5408 5490 5000
TagPicker mount 3086 3089 5000
Text mount 439 450 5000
TextField mount 1567 1534 5000
ThemeProvider mount 3323 3394 5000
ThemeProvider virtual-rerender 506 492 5000
Toggle mount 898 932 5000
ToggleNext mount 893 896 5000
button mount 124 128 5000

Perf Analysis (Fluent)

⚠️ 5 potential perf regressions detected

Potential regressions comparing to master

Scenario Current PR Ticks Baseline Ticks Ratio Regression Analysis
ButtonOverridesMissPerf.default 133 55 2.42:1 analysis
ButtonUseCssNestingPerf.default 67 51 1.31:1 analysis
ButtonUseCssPerf.default 62 54 1.15:1 analysis
ListCommonPerf.default 740 1078 0.69:1 analysis
ListNestedPerf.default 664 975 0.68:1 analysis
Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🎯 Avatar.Fluent 0.51 0.53 0.96:1 2000 1013
🦄 Button.Fluent 0.12 0.21 0.57:1 5000 592
🔧 Checkbox.Fluent 0.78 0.37 2.11:1 1000 775
🎯 Dialog.Fluent 0.17 0.24 0.71:1 5000 832
🔧 Dropdown.Fluent 3.34 0.52 6.42:1 1000 3339
🔧 Icon.Fluent 0.15 0.06 2.5:1 5000 756
🦄 Image.Fluent 0.08 0.12 0.67:1 5000 423
🔧 Slider.Fluent 1.85 0.39 4.74:1 1000 1849
🔧 Text.Fluent 0.07 0.03 2.33:1 5000 366
🦄 Tooltip.Fluent 0.12 21.87 0.01:1 5000 586

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
FlexMinimalPerf.default 329 282 1.17:1
RefMinimalPerf.default 252 217 1.16:1
AccordionMinimalPerf.default 167 150 1.11:1
ButtonMinimalPerf.default 203 183 1.11:1
SegmentMinimalPerf.default 380 341 1.11:1
AnimationMinimalPerf.default 431 398 1.08:1
ImageMinimalPerf.default 401 373 1.08:1
ButtonSlotsPerf.default 701 654 1.07:1
TextAreaMinimalPerf.default 507 472 1.07:1
Image.Fluent 423 395 1.07:1
DialogMinimalPerf.default 855 803 1.06:1
GridMinimalPerf.default 360 341 1.06:1
ProviderMergeThemesPerf.default 2222 2092 1.06:1
ProviderMinimalPerf.default 1141 1072 1.06:1
AttachmentMinimalPerf.default 180 172 1.05:1
ChatMinimalPerf.default 664 635 1.05:1
TextMinimalPerf.default 376 357 1.05:1
ToolbarMinimalPerf.default 1064 1011 1.05:1
Text.Fluent 366 348 1.05:1
FormMinimalPerf.default 430 414 1.04:1
LabelMinimalPerf.default 443 426 1.04:1
LayoutMinimalPerf.default 426 411 1.04:1
TableManyItemsPerf.default 2513 2406 1.04:1
Button.Fluent 592 571 1.04:1
Tooltip.Fluent 586 565 1.04:1
AlertMinimalPerf.default 334 323 1.03:1
AvatarMinimalPerf.default 540 523 1.03:1
RadioGroupMinimalPerf.default 459 444 1.03:1
SplitButtonMinimalPerf.default 4358 4224 1.03:1
CustomToolbarPrototype.default 4409 4300 1.03:1
TooltipMinimalPerf.default 864 839 1.03:1
VideoMinimalPerf.default 709 687 1.03:1
Checkbox.Fluent 775 750 1.03:1
ChatDuplicateMessagesPerf.default 482 473 1.02:1
ListMinimalPerf.default 503 494 1.02:1
LoaderMinimalPerf.default 842 823 1.02:1
MenuButtonMinimalPerf.default 1694 1653 1.02:1
PopupMinimalPerf.default 744 731 1.02:1
ReactionMinimalPerf.default 422 415 1.02:1
SkeletonMinimalPerf.default 432 422 1.02:1
Avatar.Fluent 1013 992 1.02:1
Dialog.Fluent 832 814 1.02:1
CarouselMinimalPerf.default 520 513 1.01:1
CheckboxMinimalPerf.default 3255 3230 1.01:1
EmbedMinimalPerf.default 2154 2131 1.01:1
HeaderMinimalPerf.default 399 395 1.01:1
InputMinimalPerf.default 1488 1478 1.01:1
ItemLayoutMinimalPerf.default 1428 1420 1.01:1
PortalMinimalPerf.default 141 139 1.01:1
StatusMinimalPerf.default 724 717 1.01:1
TableMinimalPerf.default 427 421 1.01:1
Dropdown.Fluent 3339 3291 1.01:1
CardMinimalPerf.default 588 590 1:1
ChatWithPopoverPerf.default 548 549 1:1
DropdownManyItemsPerf.default 871 872 1:1
TreeMinimalPerf.default 966 970 1:1
Icon.Fluent 756 759 1:1
Slider.Fluent 1849 1857 1:1
BoxMinimalPerf.default 363 368 0.99:1
SliderMinimalPerf.default 1839 1866 0.99:1
IconMinimalPerf.default 718 725 0.99:1
HeaderSlotsPerf.default 826 845 0.98:1
DividerMinimalPerf.default 381 391 0.97:1
MenuMinimalPerf.default 925 954 0.97:1
AttachmentSlotsPerf.default 1260 1306 0.96:1
DropdownMinimalPerf.default 3344 3478 0.96:1
TreeWith60ListItems.default 235 249 0.94:1
ListWith60ListItems.default 1079 1258 0.86:1

@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 497acb3:

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

@dzearing dzearing closed this Aug 17, 2020
@dzearing dzearing reopened this Aug 17, 2020
@dzearing
Copy link
Member

Resetting build.

@size-auditor
Copy link

size-auditor bot commented Aug 17, 2020

Asset size changes

Project Bundle Baseline Size New Size Difference
office-ui-fabric-react office-ui-fabric-react-Stack 38.125 kB 38.143 kB ExceedsBaseline     18 bytes
office-ui-fabric-react fluentui-react-next-TeachingBubble 195.932 kB 195.95 kB ExceedsBaseline     18 bytes
office-ui-fabric-react office-ui-fabric-react-TeachingBubble 195.932 kB 195.95 kB ExceedsBaseline     18 bytes
office-ui-fabric-react fluentui-react-next-Stack 38.125 kB 38.143 kB ExceedsBaseline     18 bytes

ExceedsTolerance Over Tolerance (1024 B) ExceedsBaseline Over Baseline BelowBaseline Below Baseline New New Deleted  Removed 1 kB = 1000 B

Baseline commit: 0780384fb2ea7bffcad8d51df0ca8aaa7e19be9c (build)

@khmakoto
Copy link
Member

@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 Mon, 17 Aug 2020 17:50:29 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".

@msft-github-bot msft-github-bot merged commit bb0183e into microsoft:master Aug 17, 2020
@msft-github-bot
Copy link
Contributor

🎉[email protected] has been released which incorporates this pull request.:tada:

Handy links:

@pavanpadavala pavanpadavala deleted the StackItemPaddingFix branch August 22, 2020 17:12
levithomason pushed a commit to levithomason/fluentui that referenced this pull request Aug 24, 2020
…ken (microsoft#14554)

#### Pull request checklist

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

#### Description of changes

StackItem styles has a 'padding' token but it is not used in the code. so adding that to fix and function as designed.

#### 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.

StackItem does not use its padding token
4 participants