Skip to content

chore: Update Field's VR tests to test changes from FieldContext#27400

Merged
behowell merged 2 commits into
microsoft:masterfrom
behowell:field/context-vr
Apr 10, 2023
Merged

chore: Update Field's VR tests to test changes from FieldContext#27400
behowell merged 2 commits into
microsoft:masterfrom
behowell:field/context-vr

Conversation

@behowell

@behowell behowell commented Mar 31, 2023

Copy link
Copy Markdown
Contributor

Previous Behavior

Field VR tests weren't sufficiently testing the new behavior as of:

New Behavior

  • Allow controls to inherit size via the context.
  • Add additional states for most controls in a field (beyond just error).

@fabricteam

fabricteam commented Mar 31, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 1071a2a382741b1328228b336c2a94de388a3963

@size-auditor

size-auditor Bot commented Mar 31, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: af3c827937dcb0a32f52ef8a93b91c1276956b56 (build)

Comment thread apps/vr-tests-react-components/src/stories/Field.stories.tsx
@codesandbox-ci

codesandbox-ci Bot commented Apr 1, 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 b80c80e:

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

@fabricteam

Copy link
Copy Markdown
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 887 881 5000
Button mount 586 590 5000
Field mount 1556 1477 5000
FluentProvider mount 1095 1086 5000
FluentProviderWithTheme mount 290 296 10
FluentProviderWithTheme virtual-rerender 275 272 10
FluentProviderWithTheme virtual-rerender-with-unmount 293 285 10
InfoButton mount 203 206 5000
MakeStyles mount 1382 1401 50000
Persona mount 2155 2056 5000
SpinButton mount 1820 1871 5000

@behowell
behowell marked this pull request as ready for review April 5, 2023 20:09
@behowell
behowell requested a review from a team as a code owner April 5, 2023 20:09
@behowell
behowell requested a review from a team April 10, 2023 16:46
@behowell
behowell merged commit dffe6fb into microsoft:master Apr 10, 2023
@behowell
behowell deleted the field/context-vr branch April 10, 2023 18:45
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