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: implements nested tree selection #28668

Merged

Conversation

bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Jul 28, 2023

Previous Behavior

New Behavior

  1. implements nested tree controlled selection
  2. adds unit tests for flat checked tree
  3. fix initial state of uncontrolled flat checked items

Related Issue(s)

@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 c571da5:

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

@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.544 kB
18.216 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
208.091 kB
57.972 kB
react-components
react-components: FluentProvider & webLightTheme
36.409 kB
12.01 kB
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
🤖 This report was generated against 262900bd47ddaa279b97f229b213f8a0ad796535

@fabricteam
Copy link
Collaborator

fabricteam commented Jul 28, 2023

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 602 609 5000
Button mount 291 291 5000
Field mount 1098 1122 5000
FluentProvider mount 646 638 5000
FluentProviderWithTheme mount 78 81 10
FluentProviderWithTheme virtual-rerender 69 66 10
FluentProviderWithTheme virtual-rerender-with-unmount 70 68 10
InfoButton mount 12 10 5000
MakeStyles mount 856 896 50000
Persona mount 1656 1620 5000
SpinButton mount 1326 1343 5000

@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: 262900bd47ddaa279b97f229b213f8a0ad796535 (build)

@bsunderhus bsunderhus marked this pull request as ready for review July 28, 2023 11:36
@bsunderhus bsunderhus requested a review from a team as a code owner July 28, 2023 11:36
@bsunderhus bsunderhus force-pushed the react-tree/feat--nested-tree-checked branch 2 times, most recently from 5329158 to 4771ce0 Compare July 28, 2023 12:59
@bsunderhus bsunderhus added the Status: Blocked Resolution blocked by another issue label Jul 31, 2023
@bsunderhus bsunderhus force-pushed the react-tree/feat--nested-tree-checked branch from 4771ce0 to f045e48 Compare July 31, 2023 10:06
@bsunderhus bsunderhus removed the Status: Blocked Resolution blocked by another issue label Jul 31, 2023
@bsunderhus bsunderhus requested a review from ling1726 July 31, 2023 11:01
@bsunderhus bsunderhus force-pushed the react-tree/feat--nested-tree-checked branch 2 times, most recently from 2cf3e11 to 3da35ca Compare August 2, 2023 08:44
@bsunderhus bsunderhus force-pushed the react-tree/feat--nested-tree-checked branch from 3da35ca to c571da5 Compare August 2, 2023 10:50
@bsunderhus bsunderhus merged commit 253798f into microsoft:master Aug 4, 2023
@bsunderhus bsunderhus deleted the react-tree/feat--nested-tree-checked branch August 4, 2023 12:29
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 7, 2023
* master:
  chore(cxe-prg): migrate to new slot API (microsoft#28752)
  chore(teams-prg): migrate to new slot API (microsoft#28751)
  chore: re-export slot and assertSlots methods (microsoft#28755)
  Docs(react-tree): Improve docs and stories (microsoft#28741)
  feat: implements nested tree selection (microsoft#28668)
  applying package updates
  applying package updates
  feature(react-utilities): implements new slot methods (`slot` and `assertSlots`) (microsoft#28373)
  fix(TimePicker): Selection now works in locales that don't use "am"/"pm" in their time format (microsoft#28469)
  bugfix: ensures controlled state works properly + (microsoft#28665)
  feat(tokens): Add status color tokens (microsoft#28006)
  feat(tokens): Add colorBrandStroke2Contrast and colorNeutralStrokeAlpha2 (microsoft#28638)
  Made BreadcrumbButton consistent with Item and Link (microsoft#28672)
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 7, 2023
* master:
  chore(cxe-prg): migrate to new slot API (microsoft#28752)
  chore(teams-prg): migrate to new slot API (microsoft#28751)
  chore: re-export slot and assertSlots methods (microsoft#28755)
  Docs(react-tree): Improve docs and stories (microsoft#28741)
  feat: implements nested tree selection (microsoft#28668)
  applying package updates
  applying package updates
  feature(react-utilities): implements new slot methods (`slot` and `assertSlots`) (microsoft#28373)
  fix(TimePicker): Selection now works in locales that don't use "am"/"pm" in their time format (microsoft#28469)
  bugfix: ensures controlled state works properly + (microsoft#28665)
  feat(tokens): Add status color tokens (microsoft#28006)
  feat(tokens): Add colorBrandStroke2Contrast and colorNeutralStrokeAlpha2 (microsoft#28638)
  Made BreadcrumbButton consistent with Item and Link (microsoft#28672)
  fix(EventListener): do not use `defaultProps` (microsoft#28725)
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.

[Feat] react-tree: implement selection on nested Tree
3 participants