Skip to content

Docs(react-tree): Update stories and remove third-party API calls #28757

Merged
petdud merged 3 commits into
microsoft:masterfrom
petdud:react-tree-stories-update
Aug 7, 2023
Merged

Docs(react-tree): Update stories and remove third-party API calls #28757
petdud merged 3 commits into
microsoft:masterfrom
petdud:react-tree-stories-update

Conversation

@petdud

@petdud petdud commented Aug 7, 2023

Copy link
Copy Markdown
Contributor

Previous Behavior

New Behavior

This PR:

  • Mock API calls in lazy-loading and infinite-scroll storybook examples instead of using third-party
  • Update examples for Actions
  • Improve docs for Actions and best practises involving Actions/Context menu

Related Issue(s)

  • Fixes #

@petdud
petdud marked this pull request as ready for review August 7, 2023 11:48
@petdud
petdud requested review from a team as code owners August 7, 2023 11:48
@fabricteam

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender 66 68 10 Possible regression
InfoButton mount 12 13 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 649 646 5000
Button mount 312 311 5000
Field mount 1147 1154 5000
FluentProvider mount 675 686 5000
FluentProviderWithTheme mount 77 81 10
FluentProviderWithTheme virtual-rerender 66 68 10 Possible regression
FluentProviderWithTheme virtual-rerender-with-unmount 78 79 10
InfoButton mount 12 13 5000 Possible regression
MakeStyles mount 858 850 50000
Persona mount 1834 1766 5000
SpinButton mount 1445 1397 5000

@codesandbox-ci

codesandbox-ci Bot commented Aug 7, 2023

Copy link
Copy Markdown

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 38d52d6:

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

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
68.985 kB
18.63 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
208.493 kB
58.137 kB
react-components
react-components: FluentProvider & webLightTheme
37.573 kB
12.31 kB
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
react-tree
FlatTree
132.463 kB
37.564 kB
react-tree
PersonaFlatTree
133.814 kB
37.822 kB
react-tree
PersonaTree
130.749 kB
36.732 kB
react-tree
Tree
129.399 kB
36.48 kB
🤖 This report was generated against d5ece238df797e84a59affc6162b0fa5cc46649d

@size-auditor

size-auditor Bot commented Aug 7, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: d5ece238df797e84a59affc6162b0fa5cc46649d (build)

@petdud
petdud enabled auto-merge (squash) August 7, 2023 12:42
@petdud
petdud requested a review from bsunderhus August 7, 2023 12:52
@petdud petdud linked an issue Aug 7, 2023 that may be closed by this pull request
@petdud
petdud merged commit cde8c56 into microsoft:master Aug 7, 2023
@petdud
petdud deleted the react-tree-stories-update branch August 7, 2023 13:41
bsunderhus added a commit to bsunderhus/fluentui that referenced this pull request Aug 8, 2023
bsunderhus added a commit that referenced this pull request Aug 10, 2023
…ufixes (#28776)

* chore: update stories according to #28757

* chore: updates documentation

* chore: properly labels stories and adds has actions

* bugfix: aria-checked should be mixed when selection is mixed

* bugfix: aria-selected is required

* chore: adds best practices to aria-selected=true

* bugfix: stop scrolling on Home and End

* chore: updates stories with ARIALive
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.

[Docs] react-tree: review all documentation

3 participants