fluentui icon indicating copy to clipboard operation
fluentui copied to clipboard

Update usePortalMountNodeStyles.styles.ts

Open layershifter opened this issue 2 years ago • 5 comments

Previous Behavior

New Behavior

Related Issue(s)

  • Fixes #

layershifter avatar Jun 02 '23 07:06 layershifter

Asset size changes

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

Baseline commit: cfacaff0d52a2d226b2ac4935f35d6608089cd01 (build)

size-auditor[bot] avatar Jun 02 '23 07:06 size-auditor[bot]

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
FluentProviderWithTheme virtual-rerender-with-unmount 66 74 10 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 578 571 5000
Button mount 309 298 5000
Field mount 1008 1024 5000
FluentProvider mount 639 648 5000
FluentProviderWithTheme mount 75 87 10
FluentProviderWithTheme virtual-rerender 68 67 10
FluentProviderWithTheme virtual-rerender-with-unmount 66 74 10 Possible regression
InfoButton mount 10 19 5000
MakeStyles mount 879 876 50000
Persona mount 1621 1607 5000
SpinButton mount 1229 1279 5000

fabricteam avatar Jun 02 '23 07:06 fabricteam

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 d43676a9c69a62c20c11999c05aaf841b04da5cd:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration
compassionate-drake-xfzvv9 Issue #26579

codesandbox-ci[bot] avatar Jun 02 '23 07:06 codesandbox-ci[bot]

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-combobox
Combobox (including child components)
87.771 kB
28.249 kB
87.915 kB
28.321 kB
144 B
72 B
react-combobox
Dropdown (including child components)
86.11 kB
27.854 kB
86.254 kB
27.93 kB
144 B
76 B
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
211.151 kB
58.715 kB
211.295 kB
58.801 kB
144 B
86 B
react-datepicker-compat
DatePicker Compat
222.624 kB
59.214 kB
222.768 kB
59.29 kB
144 B
76 B
react-infobutton
InfoButton
132.502 kB
40.405 kB
132.646 kB
40.467 kB
144 B
62 B
react-infobutton
InfoLabel
135.967 kB
41.466 kB
136.111 kB
41.528 kB
144 B
62 B
react-menu
Menu (including children components)
133.229 kB
40.538 kB
133.373 kB
40.617 kB
144 B
79 B
react-menu
Menu (including selectable components)
136.213 kB
41.069 kB
136.357 kB
41.146 kB
144 B
77 B
react-popover
Popover
119.464 kB
36.745 kB
119.608 kB
36.799 kB
144 B
54 B
react-portal
Portal
11.676 kB
4.31 kB
11.82 kB
4.391 kB
144 B
81 B
react-tooltip
Tooltip
47.119 kB
16.528 kB
47.263 kB
16.585 kB
144 B
57 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-alert
Alert
93.624 kB
22.563 kB
react-avatar
Avatar
57.872 kB
15.115 kB
react-avatar
AvatarGroup
15.682 kB
6.306 kB
react-avatar
AvatarGroupItem
74.048 kB
19.605 kB
react-components
react-components: Button, FluentProvider & webLightTheme
64.935 kB
17.918 kB
react-components
react-components: FluentProvider & webLightTheme
36.168 kB
11.964 kB
react-dialog
Dialog (including children components)
92.16 kB
27.502 kB
react-overflow
hooks only
11.206 kB
4.266 kB
react-persona
Persona
64.793 kB
17.036 kB
react-portal-compat
PortalCompatProvider
6.473 kB
2.196 kB
react-table
DataGrid
150.941 kB
41.535 kB
react-table
Table (Primitives only)
45.147 kB
12.577 kB
react-table
Table as DataGrid
133.392 kB
34.008 kB
react-table
Table (Selection only)
79.161 kB
19.386 kB
react-table
Table (Sort only)
78.491 kB
19.195 kB
react-tags
Tag
22.04 kB
7.94 kB
🤖 This report was generated against cfacaff0d52a2d226b2ac4935f35d6608089cd01

fabricteam avatar Jun 02 '23 07:06 fabricteam

@microsoft/cxe-coastal can you please review this PR? (changes in tests)

layershifter avatar Jun 08 '23 08:06 layershifter

I'm currently getting a scroll jump back to the top of the page when I open a positioned element that gets focus when opened -- the "Trapping Focus" Popover example shows this for me (on all browsers on both macOS and Windows, regardless of opening with mouse or keyboard). Looks like it's related to removing the position: fixed thing.

Thanks for reporting @smhigley 👍 ~~I will check what is the difference to existing examples in VR tests and why it's not captured by them.~~

Edit: I fixed the problem (reverted changes to applying styles) and created a new VR test.

layershifter avatar Jun 13 '23 07:06 layershifter

:tada:@fluentui/[email protected] has been released which incorporates this pull request.:tada:

Handy links:

msft-fluent-ui-bot avatar Jun 21 '23 07:06 msft-fluent-ui-bot

:tada:@fluentui/[email protected] has been released which incorporates this pull request.:tada:

Handy links:

msft-fluent-ui-bot avatar Jun 21 '23 07:06 msft-fluent-ui-bot

:tada:@fluentui/[email protected] has been released which incorporates this pull request.:tada:

Handy links:

msft-fluent-ui-bot avatar Jun 21 '23 07:06 msft-fluent-ui-bot