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

feat(react-tree): adds openItems and checkedItems to tree callback data #28669

Conversation

bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Jul 28, 2023

Previous Behavior

New Behavior

  1. reduces the amount of HTMLElementWalker instances being created
  2. adds openItems and checkedItems to tree callback data
  3. adds checked state to tree item context
  4. fix role baseTree issue

Related Issue(s)

  • Fixes #

@github-actions github-actions bot added this to the July Project Cycle Q3 2023 milestone Jul 28, 2023
@bsunderhus bsunderhus self-assigned this Jul 28, 2023
@bsunderhus bsunderhus marked this pull request as ready for review July 28, 2023 12:45
@bsunderhus bsunderhus requested a review from a team as a code owner July 28, 2023 12:45
@fabricteam
Copy link
Collaborator

fabricteam commented Jul 28, 2023

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme mount 76 73 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 584 612 5000
Button mount 283 295 5000
Field mount 1029 1047 5000
FluentProvider mount 633 654 5000
FluentProviderWithTheme mount 76 73 10 Possible regression
FluentProviderWithTheme virtual-rerender 67 66 10
FluentProviderWithTheme virtual-rerender-with-unmount 67 71 10
InfoButton mount 12 8 5000
MakeStyles mount 842 833 50000
Persona mount 1610 1592 5000
SpinButton mount 1321 1349 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 28, 2023

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
67.576 kB
18.225 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
208.062 kB
57.968 kB
react-components
react-components: FluentProvider & webLightTheme
36.409 kB
12.003 kB
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
🤖 This report was generated against 50392ee3cdec1975dff5e496da6241d37acb11b2

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jul 28, 2023

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 e2e177a:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@bsunderhus bsunderhus force-pushed the react-tree/feat--adds-openItems-and-checkedItems-to-tree-callbacks branch from 4746128 to 010e5ad Compare July 28, 2023 13:02
@size-auditor
Copy link

size-auditor bot commented Jul 28, 2023

Asset size changes

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

Baseline commit: 50392ee3cdec1975dff5e496da6241d37acb11b2 (build)

@bsunderhus bsunderhus force-pushed the react-tree/feat--adds-openItems-and-checkedItems-to-tree-callbacks branch from 010e5ad to e2e177a Compare July 31, 2023 07:31
@bsunderhus bsunderhus requested a review from ling1726 July 31, 2023 07:32
@bsunderhus bsunderhus merged commit f9e014b into microsoft:master Jul 31, 2023
@bsunderhus bsunderhus deleted the react-tree/feat--adds-openItems-and-checkedItems-to-tree-callbacks branch July 31, 2023 09:53
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 1, 2023
* master: (37 commits)
  release (microsoft#28696)
  Fixing re-render issue for all charts when empty (microsoft#28321)
  feat(FluentProvider): emit errors on duplicate IDs (microsoft#28670)
  applying package updates
  fix(react-positioning): autoSize causing position update to reach maximum (microsoft#28689)
  fix(react-tags-preview): fix InteractionTag hover styles (microsoft#28686)
  Accordion: export AccordionHeaderProvider (microsoft#28542)
  feat(react-shared-contexts): add AnnounceContext (microsoft#28654)
  Added VR tests for Breadcrumb (microsoft#28653)
  fix(react-menu): use outline for menuItem focus ring (microsoft#28685)
  [Bug]: Tree, vertical spacing of branches and children is inconsistent (microsoft#28681)
  feaTt(react-tree): adds openItems and checkedItems to tree callback data (microsoft#28669)
  applying package updates
  Add mountNode prop to combos (microsoft#28661)
  react-tags-preview: add more vr test (microsoft#28582)
  chore: migrate to nx 16.1.4 (microsoft#28583)
  applying package updates
  chore: improves internal headless signature (microsoft#28651)
  fix: remove margin from icon when ToolbarButton is vertical (microsoft#28658)
  applying package updates
  ...
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.

3 participants