Skip to content

Conversation

@layershifter
Copy link
Member

@layershifter layershifter marked this pull request as ready for review October 25, 2023 09:36
@layershifter layershifter requested review from a team as code owners October 25, 2023 09:36
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 638 625 5000
Button mount 327 318 5000
Field mount 1141 1153 5000
FluentProvider mount 684 696 5000
FluentProviderWithTheme mount 89 89 10
FluentProviderWithTheme virtual-rerender 61 63 10
FluentProviderWithTheme virtual-rerender-with-unmount 73 76 10
MakeStyles mount 867 851 50000
Persona mount 1785 1758 5000
SpinButton mount 1411 1398 5000

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
70.002 kB
20.161 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
207.417 kB
59.289 kB
react-components
react-components: FluentProvider & webLightTheme
42.286 kB
14 kB
react-portal-compat
PortalCompatProvider
6.651 kB
2.252 kB
react-table
DataGrid
157.084 kB
43.906 kB
react-table
Table (Primitives only)
43.941 kB
13.76 kB
react-table
Table as DataGrid
130.165 kB
35.174 kB
react-table
Table (Selection only)
75.777 kB
20.477 kB
react-table
Table (Sort only)
74.384 kB
20.075 kB
🤖 This report was generated against 61d4a2910a5039af8b8509422ecfe58bc636191c

@codesandbox-ci
Copy link

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 88a9196:

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

@size-auditor
Copy link

size-auditor bot commented Oct 25, 2023

Asset size changes

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

Baseline commit: 61d4a2910a5039af8b8509422ecfe58bc636191c (build)

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@layershifter layershifter merged commit 971eb86 into microsoft:master Oct 25, 2023
@layershifter layershifter deleted the fix/spec-paths branch October 25, 2023 17:51
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 26, 2023
* master:
  chore: migrate to nx 16.10 (microsoft#29617)
  fix(react-drawer): accessibility tweaks (microsoft#29627)
  Fixing few legend colors not visible in dark mode (microsoft#29652)
  applying package updates
  Fixing dark mode bug for xAxis for Vertical Stacked bar chart (microsoft#29427)
  chore: Refactor RadioGroup stories to use Field, and remove individual storybook page for Radio (microsoft#29268)
  docs: fix description of SelectInitialValue story (microsoft#26493)
  chore: fix specs location (microsoft#29642)
  feat(scripts-task): implement initial verify-packaging task (microsoft#29595)
  feat(tools): replace npmignore with package.json#files for robust control over what we ship to npm (microsoft#29616)
  fix(workspace-plugin): fix storybook and vr-test issues wihtin prepare-initial-release generator (microsoft#29577)
  feat(react-breadcrumb): removed non-interactive item  (microsoft#29623)
  fix(react-motion): prevent from breaking the rule of hooks (microsoft#29628)
  fix(react-toast): invoke useCustomStyleHook_unstable() (microsoft#29632)
  feat: Deprecate Alert component (microsoft#29615)
  chore(react-infobutton, react-components): Deprecate InfoButton and InfoLabel from react-infobutton (microsoft#29605)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: @fluentui/react-table has spec at invalid location [Bug]: @fluentui/react-skeleton has spec at invalid location

4 participants