Skip to content

Conversation

@GeoffCoxMSFT
Copy link
Member

Changes

  • Updated components to call custom style hook

Issues

@GeoffCoxMSFT GeoffCoxMSFT self-assigned this Mar 4, 2023
@GeoffCoxMSFT GeoffCoxMSFT requested a review from a team as a code owner March 4, 2023 18:23
@github-actions github-actions bot added this to the March Project Cycle Q1 2023 milestone Mar 4, 2023
@size-auditor
Copy link

size-auditor bot commented Mar 4, 2023

Asset size changes

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

Baseline commit: 5b6822abdd478455086630785d35d464ca68a1cc (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Mar 4, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-text
Text - Default
12.261 kB
4.819 kB
14.898 kB
5.31 kB
2.637 kB
491 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
67.03 kB
18.099 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
206.273 kB
57.145 kB
react-components
react-components: FluentProvider & webLightTheme
37.861 kB
12.034 kB
react-input
Input
25.664 kB
7.695 kB
react-input
InputField
35.613 kB
10.592 kB
react-portal-compat
PortalCompatProvider
6.324 kB
2.129 kB
react-text
Text - Wrappers
15.571 kB
5.23 kB
🤖 This report was generated against 5b6822abdd478455086630785d35d464ca68a1cc

@codesandbox-ci
Copy link

codesandbox-ci bot commented Mar 4, 2023

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 36478a9:

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

@fabricteam
Copy link
Collaborator

fabricteam commented Mar 4, 2023

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1322 1332 5000
Button mount 944 943 5000
Field mount 2064 2049 5000
FluentProvider mount 1664 1702 5000
FluentProviderWithTheme mount 635 646 10
FluentProviderWithTheme virtual-rerender 592 597 10
FluentProviderWithTheme virtual-rerender-with-unmount 625 654 10
InfoButton mount 571 565 5000
MakeStyles mount 1907 1911 50000
Persona mount 3066 3047 5000
SpinButton mount 2492 2470 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Mar 4, 2023

🕵 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-text feat: Adding calls to custom style hooks derived from context in react-text components Mar 6, 2023
@tudorpopams tudorpopams requested a review from marcosmoura March 7, 2023 13:08
@GeoffCoxMSFT GeoffCoxMSFT merged commit 32e934b into microsoft:master Mar 14, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Mar 15, 2023
* master: (141 commits)
  chore: prerequisite changes before migrating v9 packages to SWC based transpilation (microsoft#26965)
  feat: Add InfoLabel component (microsoft#27118)
  fix(tools): update migrate-converged generator to add node field to package.json exports map (microsoft#27152)
  applying package updates
  fix: Embed play/pause indicator should render correctly in high contrast mode (microsoft#27213)
  feat(tokens): Add colorNeutralBackgroundAlpha and colorNeutralStrokeAlpha tokens (microsoft#27034)
  chore: Prepare release 0.66.4 (microsoft#27163)
  fix(react-datepicker-compat): Fix text entry issues with input (microsoft#27204)
  fix: Smooth out animation of indeterminate progress bar (microsoft#27201)
  Add cxe-red as a codeowner of react-file-type-icons (microsoft#27147)
  Added documentation about custom style hooks (microsoft#27087)
  Custom style react-text (microsoft#27078)
  Custom style react-image (microsoft#27066)
  Add JSDoc for tabster focus indicator selector (microsoft#27041)
  Theme designer: Move colors to local files (microsoft#27191)
  docs(react-card): move card out of preview (microsoft#27185)
  Include values in typography table (microsoft#27189)
  feat: accept a className in `mountNode` in `Portal` (microsoft#27008)
  RFC: Extend `mountNode` prop in `Portal` (microsoft#27009)
  applying package updates
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Mar 15, 2023
* master: (141 commits)
  chore: prerequisite changes before migrating v9 packages to SWC based transpilation (microsoft#26965)
  feat: Add InfoLabel component (microsoft#27118)
  fix(tools): update migrate-converged generator to add node field to package.json exports map (microsoft#27152)
  applying package updates
  fix: Embed play/pause indicator should render correctly in high contrast mode (microsoft#27213)
  feat(tokens): Add colorNeutralBackgroundAlpha and colorNeutralStrokeAlpha tokens (microsoft#27034)
  chore: Prepare release 0.66.4 (microsoft#27163)
  fix(react-datepicker-compat): Fix text entry issues with input (microsoft#27204)
  fix: Smooth out animation of indeterminate progress bar (microsoft#27201)
  Add cxe-red as a codeowner of react-file-type-icons (microsoft#27147)
  Added documentation about custom style hooks (microsoft#27087)
  Custom style react-text (microsoft#27078)
  Custom style react-image (microsoft#27066)
  Add JSDoc for tabster focus indicator selector (microsoft#27041)
  Theme designer: Move colors to local files (microsoft#27191)
  docs(react-card): move card out of preview (microsoft#27185)
  Include values in typography table (microsoft#27189)
  feat: accept a className in `mountNode` in `Portal` (microsoft#27008)
  RFC: Extend `mountNode` prop in `Portal` (microsoft#27009)
  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