Skip to content

[TreeView] Fix typing of the Multiple generic#10478

Merged
flaviendelangle merged 2 commits intomui:masterfrom
flaviendelangle:multi-select-typing
Sep 26, 2023
Merged

[TreeView] Fix typing of the Multiple generic#10478
flaviendelangle merged 2 commits intomui:masterfrom
flaviendelangle:multi-select-typing

Conversation

@flaviendelangle
Copy link
Copy Markdown
Member

@flaviendelangle flaviendelangle commented Sep 26, 2023

@flaviendelangle flaviendelangle added typescript scope: tree view Changes related to the tree view. This includes TreeView, TreeItem. labels Sep 26, 2023
@flaviendelangle flaviendelangle self-assigned this Sep 26, 2023
@flaviendelangle flaviendelangle marked this pull request as ready for review September 26, 2023 07:19
@flaviendelangle flaviendelangle mentioned this pull request Sep 26, 2023
1 task
@mui-bot
Copy link
Copy Markdown

mui-bot commented Sep 26, 2023

Netlify deploy preview

Netlify deploy preview: https://deploy-preview-10478--material-ui-x.netlify.app/

Updated pages

No updates.

These are the results for the performance tests:

Test case Unit Min Max Median Mean σ
Filter 100k rows ms -195.2 268 -71.8 -24.16 156.102
Sort 100k rows ms 759.5 1,720.1 1,463.6 1,324.88 316.795
Select 100k rows ms 733.7 900 748.1 787.88 64.392
Deselect 100k rows ms 141.6 266.2 217.5 210.42 40.007

Generated by 🚫 dangerJS against e061323

it('should warn when switching from controlled to uncontrolled of the selected prop', () => {
const { setProps } = render(
<TreeView selected={[]}>
<TreeView selected={null}>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This test was not coherent

Copy link
Copy Markdown
Member

@Janpot Janpot left a comment

Choose a reason for hiding this comment

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

@flaviendelangle flaviendelangle merged commit 720cccc into mui:master Sep 26, 2023
@flaviendelangle flaviendelangle deleted the multi-select-typing branch September 26, 2023 08:36
@Janpot Janpot mentioned this pull request Sep 26, 2023
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: tree view Changes related to the tree view. This includes TreeView, TreeItem. typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TreeView] Multiple generic not inferred when multiSelect prop is missing

3 participants