Skip to content

feat: Adding calls to custom style hooks derived from context in react-switch components#27076

Merged
GeoffCoxMSFT merged 4 commits into
microsoft:masterfrom
GeoffCoxMSFT:custom-style-switch
Mar 7, 2023
Merged

feat: Adding calls to custom style hooks derived from context in react-switch components#27076
GeoffCoxMSFT merged 4 commits into
microsoft:masterfrom
GeoffCoxMSFT:custom-style-switch

Conversation

@GeoffCoxMSFT

Copy link
Copy Markdown
Member

Changes

  • Updated components to call custom style hook

Issues

@GeoffCoxMSFT GeoffCoxMSFT self-assigned this Mar 4, 2023
@GeoffCoxMSFT GeoffCoxMSFT requested review from a team, behowell and khmakoto as code owners March 4, 2023 18:16
@github-actions github-actions Bot added this to the March Project Cycle Q1 2023 milestone Mar 4, 2023
@size-auditor

size-auditor Bot commented Mar 4, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: 606845129c288af984f0f43dc44c54f9093c39eb (build)

@fabricteam

fabricteam commented Mar 4, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-switch
Switch
28.225 kB
8.755 kB
30.981 kB
9.252 kB
2.756 kB
497 B
react-switch
SwitchField
35.01 kB
10.655 kB
37.766 kB
11.136 kB
2.756 kB
481 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
67.061 kB
18.105 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
206.05 kB
57.069 kB
react-components
react-components: FluentProvider & webLightTheme
37.891 kB
12.039 kB
react-portal-compat
PortalCompatProvider
6.324 kB
2.129 kB
🤖 This report was generated against 606845129c288af984f0f43dc44c54f9093c39eb

@codesandbox-ci

codesandbox-ci Bot commented Mar 4, 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 3257964:

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

@fabricteam

fabricteam commented Mar 4, 2023

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 1331 1333 5000
Button mount 963 953 5000
Field mount 2044 2017 5000
FluentProvider mount 1648 1650 5000
FluentProviderWithTheme mount 641 639 10
FluentProviderWithTheme virtual-rerender 590 593 10
FluentProviderWithTheme virtual-rerender-with-unmount 636 624 10
InfoButton mount 567 577 5000
MakeStyles mount 1911 1909 50000
Persona mount 3063 3031 5000
SpinButton mount 2423 2463 5000

@fabricteam

fabricteam commented Mar 4, 2023

Copy link
Copy Markdown
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@GeoffCoxMSFT GeoffCoxMSFT enabled auto-merge (squash) March 5, 2023 21:11
@khmakoto khmakoto changed the title Custom style react-switch feat: Adding calls to custom style hooks derived from context in react-switch components Mar 6, 2023
Comment thread change/@fluentui-react-switch-0c92ac0a-383a-4196-94b3-bef8db37c9e0.json Outdated
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.

3 participants