Skip to content

Conversation

@george-cz
Copy link
Contributor

Found a small bug in our docs, the default value wasn't properly selected in either example, as the default value was inconsistend with the option value.

@george-cz george-cz requested review from a team and smhigley as code owners October 30, 2023 22:21
@george-cz george-cz enabled auto-merge (squash) October 30, 2023 22:28
@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 627 639 5000
Button mount 320 320 5000
Field mount 1196 1137 5000
FluentProvider mount 698 703 5000
FluentProviderWithTheme mount 78 77 10
FluentProviderWithTheme virtual-rerender 65 60 10
FluentProviderWithTheme virtual-rerender-with-unmount 76 73 10
MakeStyles mount 866 858 50000
Persona mount 1810 1729 5000
SpinButton mount 1419 1432 5000

@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 68e3be8:

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-combobox
Combobox (including child components)
91.781 kB
29.935 kB
react-combobox
Dropdown (including child components)
90.223 kB
29.601 kB
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
🤖 This report was generated against 21232e3d593ae30391159108aaa9aab7a4991b91

@size-auditor
Copy link

size-auditor bot commented Oct 30, 2023

Asset size changes

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

Baseline commit: 21232e3d593ae30391159108aaa9aab7a4991b91 (build)

@george-cz george-cz merged commit 688efac into microsoft:master Oct 30, 2023
@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 31, 2023
* master: (34 commits)
  Remove redundant pipeline yaml. It is now replaced iwth azure-pipelines-ci.yml and azure-pipelines-pr.yml (microsoft#29706)
  chore(TimePicker-compat): preview release with added stories and docs (microsoft#29677)
  Enable 1ES PT to be compliant for all production pipelines (microsoft#29702)
  chore: restrict  node versions to 18 ||| 20 (microsoft#29676)
  chore(TimePicker-compat): renames and code polish (microsoft#29695)
  chore: remove animation from test environments (microsoft#29692)
  applying package updates
  docs: fix defaultSelectedOptions for Dropdown and Combobox examples (microsoft#29700)
  revert accidentally pushed changes (microsoft#29696)
  Added scale factor to minHeight logic (microsoft#29653)
  Fixing the mistake in useArrowNavigationGroup - axis values in the doc (microsoft#29667)
  applying package updates
  🧪
  🧪
  🧪
  🧪
  🧪
  Rever accidentally pushed pipeline changes (microsoft#29685)
  applying package updates
  🧪
  ...
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.

4 participants