Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(react-components, react-message-bar): Export missing MessageBar contextValues #31383

Merged
merged 3 commits into from
May 17, 2024

Conversation

sopranopillow
Copy link
Contributor

Previous Behavior

New Behavior

From a previous PR only ContextValue types were exported

Related Issue(s)

Exports ContextValues as well.

@sopranopillow sopranopillow self-assigned this May 15, 2024
@sopranopillow sopranopillow requested review from a team as code owners May 15, 2024 18:30
Copy link

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.

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 677 626 5000
Button mount 312 315 5000
Field mount 1158 1168 5000
FluentProvider mount 719 721 5000
FluentProviderWithTheme mount 84 86 10
FluentProviderWithTheme virtual-rerender 34 41 10
FluentProviderWithTheme virtual-rerender-with-unmount 84 78 10
MakeStyles mount 883 863 50000
Persona mount 1804 1749 5000
SpinButton mount 1407 1368 5000
SwatchPicker mount 1545 1571 5000

@fabricteam
Copy link
Collaborator

📊 Bundle size report

✅ No changes found

@sopranopillow sopranopillow merged commit a19a317 into microsoft:master May 17, 2024
20 checks passed
@sopranopillow sopranopillow deleted the messagebar-context branch May 17, 2024 05:43
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.

None yet

4 participants