Skip to content

chore(react-tree): improve ImmutableSet and ImmutableMap internal implementation#32167

Merged
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-tree/chore--Immutable-classes
Aug 6, 2024
Merged

chore(react-tree): improve ImmutableSet and ImmutableMap internal implementation#32167
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-tree/chore--Immutable-classes

Conversation

@bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Jul 31, 2024

Previous Behavior

New Behavior

  1. converts internal implementation of ImmutableSet and ImmutableMap to classes

Related Issue(s)

  • Fixes #

@bsunderhus bsunderhus self-assigned this Jul 31, 2024
@github-actions github-actions bot added this to the July Project Cycle Q3 2024 milestone Jul 31, 2024
@fabricteam
Copy link
Collaborator

fabricteam commented Jul 31, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: entire library
1.094 MB
270.277 kB
1.095 MB
270.371 kB
100 B
94 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
69.141 kB
20.157 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
217.326 kB
63.005 kB
react-components
react-components: FluentProvider & webLightTheme
44.442 kB
14.607 kB
react-portal-compat
PortalCompatProvider
8.39 kB
2.64 kB
react-timepicker-compat
TimePicker
106.751 kB
35.596 kB
🤖 This report was generated against bf7891fe2dfe3c66422f949aa5d95a1855c508be

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 31, 2024

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender 39 35 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 72 77 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 659 631 5000
Button mount 292 308 5000
Field mount 1091 1101 5000
FluentProvider mount 689 712 5000
FluentProviderWithTheme mount 84 91 10
FluentProviderWithTheme virtual-rerender 39 35 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 72 77 10 Possible regression
MakeStyles mount 839 870 50000
Persona mount 1799 1731 5000
SpinButton mount 1360 1380 5000
SwatchPicker mount 1647 1649 5000

@bsunderhus bsunderhus force-pushed the react-tree/chore--Immutable-classes branch 3 times, most recently from 775077f to 82c5986 Compare July 31, 2024 13:50
@bsunderhus bsunderhus force-pushed the react-tree/chore--Immutable-classes branch from 82c5986 to 7852a61 Compare August 1, 2024 10:37
@bsunderhus bsunderhus marked this pull request as ready for review August 1, 2024 11:23
@bsunderhus bsunderhus requested a review from a team as a code owner August 1, 2024 11:23
@bsunderhus bsunderhus merged commit 955bf54 into microsoft:master Aug 6, 2024
@bsunderhus bsunderhus deleted the react-tree/chore--Immutable-classes branch August 6, 2024 07:22
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 7, 2024
* master: (48 commits)
  chore: migrate to storybook v7 (microsoft#32018)
  release: applying package updates - react-components
  ci: make public-docsite-v9 deploy pipeline work and make it faster (microsoft#32248)
  fix(Toolbar): hardcode size of `ToolbarButton` (microsoft#32185)
  chore: deprecate @fluentui/react-motion-preview (microsoft#32190)
  release: applying package updates - web-components
  docs: add wrapping menu item example to ContextualMenu docs (microsoft#31289)
  docs: update title of v8 keyboard-accessible drag & drop example, add docs (microsoft#32216)
  release: applying package updates - react-components
  fix: Card does not override specified focusMode based on event listeners (microsoft#32200)
  Fix undefined reference in older browsers in useMessageBarReflow (microsoft#32238)
  feat: Add transparent appearance to ToolbarButton (microsoft#32205)
  fix(react-tabs): ignore ref for tab reserved space content slot (microsoft#31775)
  fix(Dialog): do not require ref forwarding (microsoft#32095)
  feat: add verify-packaging to react v8 release pipeline (microsoft#32212)
  chore(web-components): remove type-check definition and follow repo target defaults for consistency and type-check speeds (microsoft#32208)
  chore(react-tree): improve ImmutableSet and ImmutableMap internal implementation (microsoft#32167)
  release: applying package updates - web-components
  feat(web-components): add tablist (microsoft#32098)
  release: applying package updates - react-components
  ...
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.

3 participants