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

chore: Updating version of @fluentui/react-icons to ^2.0.245 #31802

Merged
merged 6 commits into from
Jun 25, 2024

Conversation

tomi-msft
Copy link
Contributor

@tomi-msft tomi-msft commented Jun 24, 2024

This PR is to upgrade @fluentui/react-icons to version ^2.0.245. There was a recent addition to the system icons repo that was a breaking change, which caused issues within other products using the icon package.

This PR reverted the change in the system-icons repo, and now this PR is to pick up the latest icons as well as the reverted changes.

@github-actions github-actions bot added this to the April Project Cycle Q1 2024 milestone Jun 24, 2024
@khmakoto khmakoto changed the title React icons 245 update chore: Updating version of @fluentui/react-icons to ^2.0.245 Jun 24, 2024
@fabricteam
Copy link
Collaborator

fabricteam commented Jun 24, 2024

📊 Bundle size report

✅ No changes found

yarn.lock Outdated Show resolved Hide resolved
@fabricteam
Copy link
Collaborator

fabricteam commented Jun 24, 2024

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 638 650 5000
Button mount 310 311 5000
Field mount 1121 1137 5000
FluentProvider mount 700 712 5000
FluentProviderWithTheme mount 73 92 10
FluentProviderWithTheme virtual-rerender 32 37 10
FluentProviderWithTheme virtual-rerender-with-unmount 74 75 10
MakeStyles mount 864 869 50000
Persona mount 1729 1677 5000
SpinButton mount 1403 1469 5000
SwatchPicker mount 1642 1752 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Jun 24, 2024

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
AttachmentMinimalPerf.default 88 77 1.14:1
CarouselMinimalPerf.default 274 247 1.11:1
CardMinimalPerf.default 319 291 1.1:1
AccordionMinimalPerf.default 82 76 1.08:1
AvatarMinimalPerf.default 110 102 1.08:1
BoxMinimalPerf.default 204 189 1.08:1
TreeWith60ListItems.default 91 84 1.08:1
ChatWithPopoverPerf.default 209 195 1.07:1
TextAreaMinimalPerf.default 291 274 1.06:1
FlexMinimalPerf.default 157 149 1.05:1
GridMinimalPerf.default 196 186 1.05:1
SkeletonMinimalPerf.default 204 195 1.05:1
TextMinimalPerf.default 207 197 1.05:1
AnimationMinimalPerf.default 309 299 1.03:1
ButtonSlotsPerf.default 319 311 1.03:1
ChatMinimalPerf.default 455 441 1.03:1
DividerMinimalPerf.default 207 200 1.03:1
InputMinimalPerf.default 543 529 1.03:1
ListMinimalPerf.default 316 307 1.03:1
ProviderMergeThemesPerf.default 659 640 1.03:1
ReactionMinimalPerf.default 213 207 1.03:1
StatusMinimalPerf.default 402 389 1.03:1
IconMinimalPerf.default 401 391 1.03:1
TooltipMinimalPerf.default 1309 1277 1.03:1
VideoMinimalPerf.default 442 430 1.03:1
ButtonOverridesMissPerf.default 679 665 1.02:1
ChatDuplicateMessagesPerf.default 155 152 1.02:1
DropdownManyItemsPerf.default 392 385 1.02:1
FormMinimalPerf.default 231 227 1.02:1
HeaderMinimalPerf.default 206 201 1.02:1
ImageMinimalPerf.default 224 219 1.02:1
ListCommonPerf.default 411 402 1.02:1
ListWith60ListItems.default 371 362 1.02:1
MenuMinimalPerf.default 505 496 1.02:1
MenuButtonMinimalPerf.default 956 934 1.02:1
RadioGroupMinimalPerf.default 261 257 1.02:1
AttachmentSlotsPerf.default 641 635 1.01:1
DropdownMinimalPerf.default 1438 1423 1.01:1
ItemLayoutMinimalPerf.default 704 699 1.01:1
RosterPerf.default 1558 1543 1.01:1
ToolbarMinimalPerf.default 555 547 1.01:1
EmbedMinimalPerf.default 1852 1846 1:1
LabelMinimalPerf.default 217 218 1:1
LayoutMinimalPerf.default 205 205 1:1
LoaderMinimalPerf.default 190 190 1:1
ProviderMinimalPerf.default 204 204 1:1
SplitButtonMinimalPerf.default 2236 2226 1:1
TreeMinimalPerf.default 480 481 1:1
AlertMinimalPerf.default 155 156 0.99:1
CheckboxMinimalPerf.default 1136 1144 0.99:1
DialogMinimalPerf.default 444 450 0.99:1
HeaderSlotsPerf.default 456 462 0.99:1
ListNestedPerf.default 330 333 0.99:1
PopupMinimalPerf.default 347 352 0.99:1
SegmentMinimalPerf.default 197 200 0.99:1
CustomToolbarPrototype.default 1475 1484 0.99:1
DatepickerMinimalPerf.default 3531 3592 0.98:1
PortalMinimalPerf.default 79 81 0.98:1
RefMinimalPerf.default 107 109 0.98:1
SliderMinimalPerf.default 741 754 0.98:1
TableManyItemsPerf.default 1089 1125 0.97:1
TableMinimalPerf.default 231 237 0.97:1
ButtonMinimalPerf.default 79 94 0.84:1

@fabricteam
Copy link
Collaborator

fabricteam commented Jun 24, 2024

🕵 FluentUIV0 No visual regressions between this PR and main

@fabricteam
Copy link
Collaborator

fabricteam commented Jun 24, 2024

Perf Analysis (@fluentui/react)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 623 630 5000
Breadcrumb mount 1691 1677 1000
Checkbox mount 1677 1706 5000
CheckboxBase mount 1491 1483 5000
ChoiceGroup mount 2976 2947 5000
ComboBox mount 666 667 1000
CommandBar mount 6552 6603 1000
ContextualMenu mount 13060 12955 1000
DefaultButton mount 789 787 5000
DetailsRow mount 2231 2238 5000
DetailsRowFast mount 2219 2236 5000
DetailsRowNoStyles mount 2055 2053 5000
Dialog mount 2759 2673 1000
DocumentCardTitle mount 234 238 1000
Dropdown mount 2018 2000 5000
FocusTrapZone mount 1143 1164 5000
FocusZone mount 1083 1069 5000
GroupedList mount 37213 41734 2
GroupedList virtual-rerender 19912 19930 2
GroupedList virtual-rerender-with-unmount 51044 50467 2
GroupedListV2 mount 221 237 2
GroupedListV2 virtual-rerender 210 217 2
GroupedListV2 virtual-rerender-with-unmount 224 232 2
IconButton mount 1131 1117 5000
Label mount 346 343 5000
Layer mount 2710 2722 5000
Link mount 386 404 5000
MenuButton mount 994 1007 5000
MessageBar mount 21765 21619 5000
Nav mount 2014 1996 1000
OverflowSet mount 802 804 5000
Panel mount 1837 1826 1000
Persona mount 733 752 1000
Pivot mount 877 943 1000
PrimaryButton mount 936 944 5000
Rating mount 4671 4710 5000
SearchBox mount 939 922 5000
Shimmer mount 1995 1962 5000
Slider mount 1349 1354 5000
SpinButton mount 2996 2962 5000
Spinner mount 403 384 5000
SplitButton mount 1860 1875 5000
Stack mount 417 420 5000
StackWithIntrinsicChildren mount 865 890 5000
StackWithTextChildren mount 2755 2744 5000
SwatchColorPicker mount 6430 6431 5000
TagPicker mount 1460 1464 5000
Text mount 398 389 5000
TextField mount 924 940 5000
ThemeProvider mount 849 866 5000
ThemeProvider virtual-rerender 580 585 5000
ThemeProvider virtual-rerender-with-unmount 1270 1290 5000
Toggle mount 599 619 5000
buttonNative mount 195 180 5000

Copy link
Contributor

@Hotell Hotell left a comment

Choose a reason for hiding this comment

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

thank you for nice PR description !

@tomi-msft tomi-msft merged commit 1e1398c into microsoft:master Jun 25, 2024
21 checks passed
@tomi-msft tomi-msft deleted the react-icons-245-update branch June 25, 2024 18:01
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 26, 2024
* master: (90 commits)
  fix(react-drawer): do not create global header/footer roles (microsoft#31799)
  release: applying package updates - react v8
  release: applying package updates - web-components
  Add 'allowParentArrowNavigation' prop to `ComboBox`. (microsoft#31805)
  Add drawer web component (microsoft#31521)
  fix: MenuButton is styled correctly when aria-expanded is a string (microsoft#31809)
  chore: Updating version of `@fluentui/react-icons` to `^2.0.245` (microsoft#31802)
  chore:(docs) Migrate e2e Cypress testing doc (microsoft#31796)
  chore:(docs) Update API Extractor file (microsoft#31651)
  chore: migrate to nx 18 (microsoft#31746)
  release: applying package updates - react v8
  release: applying package updates - web-components
  feat: update text to use element internals for custom states (microsoft#31770)
  chore(web-components): Remove define imports from story modules (microsoft#31622)
  fix(`merge-styles`): Correctly processing selectors when value is a class name and they are not wrapped in `selectors` wrapper (microsoft#31798)
  style(react-combobox): turn off react-compiler lints for non compliant code (microsoft#31795)
  docs: Add AriaLiveAnnouncer to all examples (microsoft#31794)
  fix(public-docsite-v9): disable storybook keyboard shortcuts (microsoft#31762)
  feat: V0 Attachment compact component (microsoft#31634)
  release: applying package updates - react v8
  ...
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.

9 participants