-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
fix(react-tabster): bump tabster to 7.1.4 to fix useArrowNavigationGroup with non-modal Dialog #31413
Conversation
mshoho
commented
May 17, 2024
•
edited by Hotell
edited by Hotell
- CHANGELOG https://github.com/microsoft/tabster/blob/master/CHANGELOG.md#v713
- including fix for Restorer steals focus from non-modal modals when trigger is inside of arrow navigation group tabster#374.
change/@fluentui-react-tabster-37512240-5808-4174-96b6-a8c571eead67.json
Show resolved
Hide resolved
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. |
Perf Analysis (
|
Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
---|---|---|---|---|---|
Avatar | mount | 639 | 646 | 5000 | |
Button | mount | 315 | 296 | 5000 | |
Field | mount | 1122 | 1130 | 5000 | |
FluentProvider | mount | 697 | 701 | 5000 | |
FluentProviderWithTheme | mount | 84 | 82 | 10 | |
FluentProviderWithTheme | virtual-rerender | 36 | 39 | 10 | |
FluentProviderWithTheme | virtual-rerender-with-unmount | 79 | 77 | 10 | |
MakeStyles | mount | 882 | 867 | 50000 | |
Persona | mount | 1756 | 1757 | 5000 | |
SpinButton | mount | 1366 | 1356 | 5000 | |
SwatchPicker | mount | 1550 | 1551 | 5000 |
Perf Analysis (
|
Scenario | Current PR Ticks | Baseline Ticks | Ratio |
---|---|---|---|
AvatarMinimalPerf.default | 113 | 105 | 1.08:1 |
ChatDuplicateMessagesPerf.default | 157 | 146 | 1.08:1 |
ProviderMinimalPerf.default | 207 | 192 | 1.08:1 |
AttachmentMinimalPerf.default | 85 | 80 | 1.06:1 |
ImageMinimalPerf.default | 223 | 210 | 1.06:1 |
CarouselMinimalPerf.default | 261 | 249 | 1.05:1 |
DropdownManyItemsPerf.default | 396 | 377 | 1.05:1 |
ButtonSlotsPerf.default | 319 | 306 | 1.04:1 |
DividerMinimalPerf.default | 204 | 196 | 1.04:1 |
HeaderMinimalPerf.default | 213 | 204 | 1.04:1 |
ToolbarMinimalPerf.default | 546 | 525 | 1.04:1 |
ChatMinimalPerf.default | 448 | 436 | 1.03:1 |
FlexMinimalPerf.default | 159 | 155 | 1.03:1 |
InputMinimalPerf.default | 561 | 545 | 1.03:1 |
MenuMinimalPerf.default | 524 | 508 | 1.03:1 |
PopupMinimalPerf.default | 352 | 341 | 1.03:1 |
RadioGroupMinimalPerf.default | 269 | 260 | 1.03:1 |
ReactionMinimalPerf.default | 214 | 207 | 1.03:1 |
StatusMinimalPerf.default | 390 | 380 | 1.03:1 |
TextMinimalPerf.default | 198 | 192 | 1.03:1 |
CheckboxMinimalPerf.default | 1158 | 1139 | 1.02:1 |
DatepickerMinimalPerf.default | 3737 | 3678 | 1.02:1 |
DialogMinimalPerf.default | 453 | 443 | 1.02:1 |
LabelMinimalPerf.default | 224 | 219 | 1.02:1 |
ListCommonPerf.default | 402 | 395 | 1.02:1 |
ListMinimalPerf.default | 312 | 305 | 1.02:1 |
SegmentMinimalPerf.default | 196 | 192 | 1.02:1 |
TableMinimalPerf.default | 226 | 222 | 1.02:1 |
VideoMinimalPerf.default | 440 | 431 | 1.02:1 |
AccordionMinimalPerf.default | 87 | 86 | 1.01:1 |
AnimationMinimalPerf.default | 304 | 302 | 1.01:1 |
AttachmentSlotsPerf.default | 626 | 621 | 1.01:1 |
GridMinimalPerf.default | 190 | 189 | 1.01:1 |
ListNestedPerf.default | 327 | 324 | 1.01:1 |
SkeletonMinimalPerf.default | 205 | 203 | 1.01:1 |
SliderMinimalPerf.default | 761 | 754 | 1.01:1 |
SplitButtonMinimalPerf.default | 2290 | 2258 | 1.01:1 |
CustomToolbarPrototype.default | 1492 | 1477 | 1.01:1 |
BoxMinimalPerf.default | 187 | 187 | 1:1 |
DropdownMinimalPerf.default | 1409 | 1410 | 1:1 |
ListWith60ListItems.default | 373 | 372 | 1:1 |
RefMinimalPerf.default | 110 | 110 | 1:1 |
TooltipMinimalPerf.default | 1257 | 1255 | 1:1 |
TreeMinimalPerf.default | 480 | 481 | 1:1 |
ButtonMinimalPerf.default | 86 | 87 | 0.99:1 |
CardMinimalPerf.default | 303 | 307 | 0.99:1 |
ChatWithPopoverPerf.default | 193 | 194 | 0.99:1 |
EmbedMinimalPerf.default | 1919 | 1929 | 0.99:1 |
ItemLayoutMinimalPerf.default | 700 | 705 | 0.99:1 |
MenuButtonMinimalPerf.default | 945 | 958 | 0.99:1 |
ProviderMergeThemesPerf.default | 634 | 641 | 0.99:1 |
IconMinimalPerf.default | 388 | 393 | 0.99:1 |
TableManyItemsPerf.default | 1116 | 1123 | 0.99:1 |
TextAreaMinimalPerf.default | 279 | 282 | 0.99:1 |
FormMinimalPerf.default | 226 | 231 | 0.98:1 |
LoaderMinimalPerf.default | 190 | 194 | 0.98:1 |
PortalMinimalPerf.default | 80 | 82 | 0.98:1 |
AlertMinimalPerf.default | 160 | 165 | 0.97:1 |
ButtonOverridesMissPerf.default | 648 | 665 | 0.97:1 |
HeaderSlotsPerf.default | 470 | 488 | 0.96:1 |
LayoutMinimalPerf.default | 199 | 207 | 0.96:1 |
RosterPerf.default | 1497 | 1591 | 0.94:1 |
TreeWith60ListItems.default | 85 | 93 | 0.91:1 |
change/@fluentui-react-tabster-37512240-5808-4174-96b6-a8c571eead67.json
Show resolved
Hide resolved
🕵 FluentUIV0 No visual regressions between this PR and main |
📊 Bundle size reportUnchanged fixtures
|
Perf Analysis (
|
Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
---|---|---|---|---|---|
BaseButton | mount | 644 | 629 | 5000 | |
Breadcrumb | mount | 1733 | 1711 | 1000 | |
Checkbox | mount | 1702 | 1738 | 5000 | |
CheckboxBase | mount | 1515 | 1518 | 5000 | |
ChoiceGroup | mount | 3034 | 3047 | 5000 | |
ComboBox | mount | 664 | 691 | 1000 | |
CommandBar | mount | 6858 | 6733 | 1000 | |
ContextualMenu | mount | 16419 | 16712 | 1000 | |
DefaultButton | mount | 807 | 817 | 5000 | |
DetailsRow | mount | 2255 | 2280 | 5000 | |
DetailsRowFast | mount | 2277 | 2259 | 5000 | |
DetailsRowNoStyles | mount | 2071 | 2050 | 5000 | |
Dialog | mount | 2684 | 2659 | 1000 | |
DocumentCardTitle | mount | 238 | 244 | 1000 | |
Dropdown | mount | 2029 | 2043 | 5000 | |
FocusTrapZone | mount | 1172 | 1152 | 5000 | |
FocusZone | mount | 1126 | 1102 | 5000 | |
GroupedList | mount | 42921 | 43514 | 2 | |
GroupedList | virtual-rerender | 20553 | 20479 | 2 | |
GroupedList | virtual-rerender-with-unmount | 54606 | 52665 | 2 | |
GroupedListV2 | mount | 226 | 236 | 2 | |
GroupedListV2 | virtual-rerender | 221 | 223 | 2 | |
GroupedListV2 | virtual-rerender-with-unmount | 238 | 244 | 2 | |
IconButton | mount | 1138 | 1118 | 5000 | |
Label | mount | 353 | 353 | 5000 | |
Layer | mount | 2797 | 2818 | 5000 | |
Link | mount | 394 | 404 | 5000 | |
MenuButton | mount | 1010 | 998 | 5000 | |
MessageBar | mount | 21662 | 21508 | 5000 | |
Nav | mount | 2080 | 2082 | 1000 | |
OverflowSet | mount | 800 | 808 | 5000 | |
Panel | mount | 1854 | 1780 | 1000 | |
Persona | mount | 743 | 785 | 1000 | |
Pivot | mount | 937 | 892 | 1000 | |
PrimaryButton | mount | 926 | 961 | 5000 | |
Rating | mount | 4678 | 4715 | 5000 | |
SearchBox | mount | 940 | 935 | 5000 | |
Shimmer | mount | 1952 | 1944 | 5000 | |
Slider | mount | 1356 | 1392 | 5000 | |
SpinButton | mount | 2989 | 3040 | 5000 | |
Spinner | mount | 407 | 404 | 5000 | |
SplitButton | mount | 1877 | 1855 | 5000 | |
Stack | mount | 400 | 426 | 5000 | |
StackWithIntrinsicChildren | mount | 845 | 873 | 5000 | |
StackWithTextChildren | mount | 2707 | 2687 | 5000 | |
SwatchColorPicker | mount | 6440 | 6494 | 5000 | |
TagPicker | mount | 1480 | 1524 | 5000 | |
Text | mount | 383 | 373 | 5000 | |
TextField | mount | 925 | 955 | 5000 | |
ThemeProvider | mount | 865 | 872 | 5000 | |
ThemeProvider | virtual-rerender | 580 | 595 | 5000 | |
ThemeProvider | virtual-rerender-with-unmount | 1307 | 1279 | 5000 | |
Toggle | mount | 641 | 631 | 5000 | |
buttonNative | mount | 197 | 202 | 5000 |
change/@fluentui-react-tabster-37512240-5808-4174-96b6-a8c571eead67.json
Outdated
Show resolved
Hide resolved
@mshoho I updated PR title and PR description to follow our practices. Please follow that pattern next time. ty |
@mshoho are you sure that PR of yours is making sure that everything else works as before ? that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need author feedback
…ead67.json Co-authored-by: Martin Hochel <hochelmartin@gmail.com>
…se tests are green.
Thanks for the good catch. I have removed |
🕵 fluentuiv9 No visual regressions between this PR and main |