Skip to content

fix(react-message-bar): keep motion ref out of base hook - #36549

Open
Dmytro Kirpa (dmytrokirpa) wants to merge 1 commit into
microsoft:masterfrom
dmytrokirpa:fix/react-message-bar-base-motion-ref
Open

fix(react-message-bar): keep motion ref out of base hook#36549
Dmytro Kirpa (dmytrokirpa) wants to merge 1 commit into
microsoft:masterfrom
dmytrokirpa:fix/react-message-bar-base-motion-ref

Conversation

@dmytrokirpa

@dmytrokirpa Dmytro Kirpa (dmytrokirpa) commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Fixes issue discovered in #36544

Summary

  • remove motion-specific ref handling from the headless MessageBar base hook
  • merge the motion ref in the styled MessageBar hook instead

Validation

  • yarn nx run react-message-bar:build --skipNxCache

@github-actions

Copy link
Copy Markdown

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-components
react-components: entire library
1.296 MB
326.636 kB
1.296 MB
326.637 kB
4 B
1 B
react-headless-components-preview
react-headless-components-preview: entire library
256.53 kB
74.32 kB
256.372 kB
74.282 kB
-158 B
-38 B
react-message-bar
MessageBar (all components)
24.114 kB
8.524 kB
24.121 kB
8.525 kB
7 B
1 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
66.297 kB
19.006 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
226.106 kB
68.067 kB
react-components
react-components: FluentProvider & webLightTheme
39.52 kB
13.116 kB
react-headless-components-preview
@fluentui/react-headless-components-preview/tag-picker
62.147 kB
20.587 kB
react-headless-components-preview
@fluentui/react-headless-components-preview/teaching-popover
35.416 kB
11.745 kB
react-portal-compat
PortalCompatProvider
5.341 kB
2.146 kB
react-timepicker-compat
TimePicker
140.991 kB
46.069 kB
🤖 This report was generated against 0af6d9f6b915559ea0b874e687287b953fe5e773

@github-actions

Copy link
Copy Markdown

Pull request demo site: URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants