Skip to content

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

Merged
GeoffCoxMSFT merged 5 commits into
microsoft:masterfrom
GeoffCoxMSFT:custom-style-tooltip
Mar 6, 2023
Merged

feat: Adding calls to custom style hooks derived from context in react-tooltip components#27080
GeoffCoxMSFT merged 5 commits into
microsoft:masterfrom
GeoffCoxMSFT:custom-style-tooltip

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:29
@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: 9a675edceb02ebc4353583f85e8578854935d494 (build)

@fabricteam

fabricteam commented Mar 4, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-alert
Alert
90.375 kB
22.112 kB
90.498 kB
22.113 kB
123 B
1 B
react-avatar
Avatar
51.346 kB
14.062 kB
54.18 kB
14.574 kB
2.834 kB
512 B
react-avatar
AvatarGroup
15.42 kB
6.167 kB
18.094 kB
6.67 kB
2.674 kB
503 B
react-avatar
AvatarGroupItem
67.568 kB
18.571 kB
70.456 kB
19.08 kB
2.888 kB
509 B
react-persona
Persona
58.361 kB
16.009 kB
61.195 kB
16.525 kB
2.834 kB
516 B
react-tooltip
Tooltip
46.328 kB
16.212 kB
49.085 kB
16.725 kB
2.757 kB
513 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.047 kB
57.067 kB
react-components
react-components: FluentProvider & webLightTheme
37.891 kB
12.039 kB
react-portal-compat
PortalCompatProvider
6.324 kB
2.129 kB
react-table
DataGrid
147.984 kB
40.019 kB
react-table
Table (Primitives only)
46.828 kB
12.563 kB
react-table
Table as DataGrid
137.585 kB
35.135 kB
react-table
Table (Selection only)
85.413 kB
21.134 kB
react-table
Table (Sort only)
84.734 kB
20.944 kB
🤖 This report was generated against ce1a8ce417d71d3063101bfc1dd6b8e09ee38650

@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 904925e:

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 1314 1338 5000
Button mount 939 950 5000
Field mount 2075 2024 5000
FluentProvider mount 1645 1624 5000
FluentProviderWithTheme mount 631 631 10
FluentProviderWithTheme virtual-rerender 590 600 10
FluentProviderWithTheme virtual-rerender-with-unmount 630 630 10
InfoButton mount 560 568 5000
MakeStyles mount 1894 1880 50000
Persona mount 3023 3014 5000
SpinButton mount 2467 2451 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-tooltip feat: Adding calls to custom style hooks derived from context in react-tooltip components Mar 6, 2023
Comment thread change/@fluentui-react-tooltip-2a113c58-5352-438c-8cdc-860e4357ad43.json Outdated
Comment thread packages/react-components/react-tooltip/src/components/Tooltip/Tooltip.tsx 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