Skip to content

fix(TreeItemLayout): Actions should not unmount between successive mouse events#32477

Merged
ling1726 merged 4 commits intomicrosoft:masterfrom
ling1726:react-tree/fix/actions-visibility
Sep 10, 2024
Merged

fix(TreeItemLayout): Actions should not unmount between successive mouse events#32477
ling1726 merged 4 commits intomicrosoft:masterfrom
ling1726:react-tree/fix/actions-visibility

Conversation

@ling1726
Copy link
Contributor

@ling1726 ling1726 commented Sep 6, 2024

Fixes #32476

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 6, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: entire library
1.099 MB
271.641 kB
1.099 MB
271.664 kB
66 B
23 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
69.14 kB
20.137 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
218.197 kB
63.203 kB
react-components
react-components: FluentProvider & webLightTheme
44.447 kB
14.59 kB
react-portal-compat
PortalCompatProvider
8.39 kB
2.64 kB
react-timepicker-compat
TimePicker
107.387 kB
35.758 kB
🤖 This report was generated against 32c8cb74115bd3294f48f68719b29f0232b1b1a8

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 6, 2024

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 644 626 5000
Button mount 323 316 5000
Field mount 1156 1166 5000
FluentProvider mount 695 733 5000
FluentProviderWithTheme mount 85 97 10
FluentProviderWithTheme virtual-rerender 35 43 10
FluentProviderWithTheme virtual-rerender-with-unmount 84 83 10
MakeStyles mount 866 870 50000
Persona mount 1768 1767 5000
SpinButton mount 1415 1396 5000
SwatchPicker mount 1662 1661 5000

@bsunderhus bsunderhus marked this pull request as ready for review September 9, 2024 14:25
@bsunderhus bsunderhus requested a review from a team as a code owner September 9, 2024 14:25
@ling1726 ling1726 merged commit ee0406b into microsoft:master Sep 10, 2024
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 12, 2024
* master: (77 commits)
  fix(react-drawer): update scroll state when children changes (microsoft#32818)
  feat(react-storybook-addon): improve addon to more readable names (microsoft#32815)
  chore: cleanup react-carousel-preview (microsoft#32475)
  feat(storybook): add rtl/ltr toggle storybook addon (microsoft#32814)
  Carousel: Storybook updates and fixing exports/focus (microsoft#32457)
  release: applying package updates - react v8
  release: applying package updates - web-components
  Update d3 dependency versions to 3.x.x and 4.x.x (microsoft#32463)
  RFC: Extended Design Tokens for Fluent UI React (microsoft#32058)
  update doc to reflect setTheme function change (microsoft#32490)
  fix (react-dialog): Use consistent rounding for clientHeight and innerHeight (microsoft#32480)
  fix(public-doscite-v9): global styles should not be applied to story elements (microsoft#32472)
  feat(workspace-plugin): implement verify-packaging executor (microsoft#32403)
  release: applying package updates - react-components
  Add strokeDasharray property when optimizeLargeData is true (microsoft#32494)
  fix(TreeItemLayout): Actions should not unmount between successive mouse events (microsoft#32477)
  release: applying package updates - react v8
  bugfix(react-tree): recover from tabIndex=-1 when TreeItem is removed (microsoft#32442)
  Fix onResolveSuggestions not being called after component is remounted in react 18 strict mode (microsoft#28227)
  fix(codeowners): update most packages owned by cxe-red with cxe-prg (microsoft#32445)
  ...
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.

TreeItemLayout: Umounts actions between mouse events

3 participants