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 17:22
@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)

@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 3e3f280:

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

@fabricteam
Copy link
Collaborator

fabricteam commented Mar 4, 2023

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
206.273 kB
57.145 kB
206.317 kB
57.147 kB
44 B
2 B
react-image
Image
11.259 kB
4.473 kB
13.897 kB
4.974 kB
2.638 kB
501 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: FluentProvider & webLightTheme
37.861 kB
12.034 kB
react-portal-compat
PortalCompatProvider
6.324 kB
2.129 kB
🤖 This report was generated against 5b6822abdd478455086630785d35d464ca68a1cc

@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 1305 1314 5000
Button mount 929 929 5000
Field mount 1975 1959 5000
FluentProvider mount 1571 1545 5000
FluentProviderWithTheme mount 586 575 10
FluentProviderWithTheme virtual-rerender 544 550 10
FluentProviderWithTheme virtual-rerender-with-unmount 581 588 10
InfoButton mount 521 515 5000
MakeStyles mount 1961 1949 50000
Persona mount 2913 2846 5000
SpinButton mount 2291 2291 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:10
@khmakoto khmakoto changed the title Custom style react-image feat: Adding calls to custom style hooks derived from context in react-image components Mar 6, 2023
@tudorpopams tudorpopams requested a review from marcosmoura March 7, 2023 13:09
@GeoffCoxMSFT GeoffCoxMSFT requested a review from Hotell March 9, 2023 01:24
@GeoffCoxMSFT GeoffCoxMSFT merged commit d0c590d 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