-
Notifications
You must be signed in to change notification settings - Fork 2.9k
chore(a11y-testing): Remove dependency on keyboard-keys
#19840
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
Conversation
Clean up dependency separation between v9 and non-v9 packages in the repo
|
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 e30e965:
|
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: d05b3db7f100423b47d782ddf457252171395de6 (build) |
📊 Bundle size reportUnchanged fixtures
|
Perf Analysis (
|
| Scenario | Render type | Master Ticks | PR Ticks | Iterations | Status |
|---|---|---|---|---|---|
| Avatar | mount | 1024 | 1003 | 5000 | |
| BaseButton | mount | 987 | 985 | 5000 | |
| Breadcrumb | mount | 2751 | 2803 | 1000 | |
| ButtonNext | mount | 477 | 492 | 5000 | |
| Checkbox | mount | 1672 | 1699 | 5000 | |
| CheckboxBase | mount | 1401 | 1448 | 5000 | |
| ChoiceGroup | mount | 5158 | 5191 | 5000 | |
| ComboBox | mount | 1049 | 1092 | 1000 | |
| CommandBar | mount | 10574 | 10661 | 1000 | |
| ContextualMenu | mount | 6927 | 6758 | 1000 | |
| DefaultButton | mount | 1244 | 1208 | 5000 | |
| DetailsRow | mount | 3937 | 3962 | 5000 | |
| DetailsRowFast | mount | 4019 | 4045 | 5000 | |
| DetailsRowNoStyles | mount | 3801 | 3834 | 5000 | |
| Dialog | mount | 2628 | 2523 | 1000 | |
| DocumentCardTitle | mount | 155 | 154 | 1000 | |
| Dropdown | mount | 3509 | 3495 | 5000 | |
| FluentProviderNext | mount | 7229 | 7456 | 5000 | |
| FluentProviderWithTheme | mount | 346 | 340 | 10 | |
| FluentProviderWithTheme | virtual-rerender | 94 | 98 | 10 | |
| FluentProviderWithTheme | virtual-rerender-with-unmount | 495 | 478 | 10 | |
| FocusTrapZone | mount | 1908 | 1905 | 5000 | |
| FocusZone | mount | 1880 | 1833 | 5000 | |
| IconButton | mount | 1914 | 1902 | 5000 | |
| Label | mount | 368 | 367 | 5000 | |
| Layer | mount | 3143 | 3228 | 5000 | |
| Link | mount | 498 | 518 | 5000 | |
| MakeStyles | mount | 1898 | 1880 | 50000 | |
| MenuButton | mount | 1610 | 1550 | 5000 | |
| MessageBar | mount | 2104 | 2154 | 5000 | |
| Nav | mount | 3609 | 3568 | 1000 | |
| OverflowSet | mount | 1160 | 1161 | 5000 | |
| Panel | mount | 2498 | 2441 | 1000 | |
| Persona | mount | 904 | 910 | 1000 | |
| Pivot | mount | 1498 | 1519 | 1000 | |
| PrimaryButton | mount | 1368 | 1377 | 5000 | |
| Rating | mount | 8440 | 8599 | 5000 | |
| SearchBox | mount | 1465 | 1447 | 5000 | |
| Shimmer | mount | 2745 | 2856 | 5000 | |
| Slider | mount | 2068 | 2099 | 5000 | |
| SpinButton | mount | 5322 | 5373 | 5000 | |
| Spinner | mount | 444 | 436 | 5000 | |
| SplitButton | mount | 3349 | 3303 | 5000 | |
| Stack | mount | 534 | 529 | 5000 | |
| StackWithIntrinsicChildren | mount | 1723 | 1720 | 5000 | |
| StackWithTextChildren | mount | 5118 | 5128 | 5000 | |
| SwatchColorPicker | mount | 11223 | 11050 | 5000 | |
| Tabs | mount | 1515 | 1483 | 1000 | |
| TagPicker | mount | 2735 | 2829 | 5000 | |
| TeachingBubble | mount | 13710 | 13905 | 5000 | |
| Text | mount | 487 | 483 | 5000 | |
| TextField | mount | 1543 | 1537 | 5000 | |
| ThemeProvider | mount | 1285 | 1296 | 5000 | |
| ThemeProvider | virtual-rerender | 653 | 650 | 5000 | |
| ThemeProvider | virtual-rerender-with-unmount | 2065 | 2087 | 5000 | |
| Toggle | mount | 889 | 929 | 5000 | |
| buttonNative | mount | 123 | 113 | 5000 |
Perf Analysis (@fluentui/react-northstar)
Perf tests with no regressions
| Scenario | Current PR Ticks | Baseline Ticks | Ratio |
|---|---|---|---|
| PortalMinimalPerf.default | 198 | 180 | 1.1:1 |
| BoxMinimalPerf.default | 407 | 373 | 1.09:1 |
| AvatarMinimalPerf.default | 231 | 214 | 1.08:1 |
| GridMinimalPerf.default | 383 | 353 | 1.08:1 |
| ListWith60ListItems.default | 736 | 684 | 1.08:1 |
| RadioGroupMinimalPerf.default | 520 | 482 | 1.08:1 |
| ChatWithPopoverPerf.default | 400 | 375 | 1.07:1 |
| TextMinimalPerf.default | 392 | 368 | 1.07:1 |
| TreeWith60ListItems.default | 195 | 183 | 1.07:1 |
| ProviderMinimalPerf.default | 1097 | 1049 | 1.05:1 |
| RefMinimalPerf.default | 238 | 227 | 1.05:1 |
| StatusMinimalPerf.default | 765 | 730 | 1.05:1 |
| TableMinimalPerf.default | 457 | 435 | 1.05:1 |
| TreeMinimalPerf.default | 897 | 853 | 1.05:1 |
| AnimationMinimalPerf.default | 450 | 431 | 1.04:1 |
| DropdownMinimalPerf.default | 3341 | 3211 | 1.04:1 |
| FormMinimalPerf.default | 469 | 453 | 1.04:1 |
| HeaderSlotsPerf.default | 870 | 836 | 1.04:1 |
| LayoutMinimalPerf.default | 418 | 401 | 1.04:1 |
| ReactionMinimalPerf.default | 431 | 414 | 1.04:1 |
| IconMinimalPerf.default | 712 | 687 | 1.04:1 |
| TableManyItemsPerf.default | 2164 | 2084 | 1.04:1 |
| AttachmentSlotsPerf.default | 1209 | 1173 | 1.03:1 |
| ButtonSlotsPerf.default | 603 | 583 | 1.03:1 |
| ChatMinimalPerf.default | 723 | 705 | 1.03:1 |
| DropdownManyItemsPerf.default | 782 | 761 | 1.03:1 |
| FlexMinimalPerf.default | 310 | 300 | 1.03:1 |
| ProviderMergeThemesPerf.default | 1777 | 1719 | 1.03:1 |
| AlertMinimalPerf.default | 313 | 308 | 1.02:1 |
| AttachmentMinimalPerf.default | 180 | 176 | 1.02:1 |
| ListCommonPerf.default | 710 | 698 | 1.02:1 |
| LoaderMinimalPerf.default | 750 | 733 | 1.02:1 |
| SliderMinimalPerf.default | 1711 | 1673 | 1.02:1 |
| CustomToolbarPrototype.default | 4174 | 4102 | 1.02:1 |
| AccordionMinimalPerf.default | 163 | 162 | 1.01:1 |
| ButtonMinimalPerf.default | 186 | 184 | 1.01:1 |
| CarouselMinimalPerf.default | 496 | 493 | 1.01:1 |
| DialogMinimalPerf.default | 826 | 820 | 1.01:1 |
| EmbedMinimalPerf.default | 4411 | 4363 | 1.01:1 |
| LabelMinimalPerf.default | 441 | 435 | 1.01:1 |
| MenuMinimalPerf.default | 978 | 973 | 1.01:1 |
| MenuButtonMinimalPerf.default | 1824 | 1808 | 1.01:1 |
| PopupMinimalPerf.default | 626 | 617 | 1.01:1 |
| SegmentMinimalPerf.default | 384 | 380 | 1.01:1 |
| SplitButtonMinimalPerf.default | 4577 | 4554 | 1.01:1 |
| ToolbarMinimalPerf.default | 1021 | 1006 | 1.01:1 |
| ButtonOverridesMissPerf.default | 1789 | 1790 | 1:1 |
| TooltipMinimalPerf.default | 1105 | 1106 | 1:1 |
| CardMinimalPerf.default | 594 | 603 | 0.99:1 |
| DatepickerMinimalPerf.default | 5778 | 5833 | 0.99:1 |
| DividerMinimalPerf.default | 405 | 409 | 0.99:1 |
| HeaderMinimalPerf.default | 385 | 387 | 0.99:1 |
| InputMinimalPerf.default | 1333 | 1342 | 0.99:1 |
| ItemLayoutMinimalPerf.default | 1334 | 1352 | 0.99:1 |
| RosterPerf.default | 1346 | 1360 | 0.99:1 |
| SkeletonMinimalPerf.default | 394 | 396 | 0.99:1 |
| TextAreaMinimalPerf.default | 553 | 556 | 0.99:1 |
| CheckboxMinimalPerf.default | 2837 | 2882 | 0.98:1 |
| ListNestedPerf.default | 605 | 619 | 0.98:1 |
| VideoMinimalPerf.default | 689 | 703 | 0.98:1 |
| ChatDuplicateMessagesPerf.default | 325 | 334 | 0.97:1 |
| ListMinimalPerf.default | 560 | 578 | 0.97:1 |
| ImageMinimalPerf.default | 410 | 425 | 0.96:1 |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
|
🎉 Handy links: |
…19840) Clean up dependency separation between v9 and non-v9 packages in the repo
Pull request checklist
$ yarn changeDescription of changes
Clean up dependency separation between v9 and non-v9 packages in the
repo.
The usage of the
keyboard-keyspackage was fairly limited and the string keys are self explanatory. Thekeyboard-keyspackage was intended to used in the web platform to reduce bundle size from duplicated strings.Focus areas to test
(optional)