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

chore(a11y-testing): Remove dependency on keyboard-keys #19840

Merged
merged 1 commit into from
Sep 17, 2021

Conversation

ling1726
Copy link
Member

@ling1726 ling1726 commented Sep 16, 2021

Pull request checklist

Description of changes

Clean up dependency separation between v9 and non-v9 packages in the
repo.

The usage of the keyboard-keys package was fairly limited and the string keys are self explanatory. The keyboard-keys package was intended to used in the web platform to reduce bundle size from duplicated strings.

Focus areas to test

(optional)

Clean up dependency separation between v9 and non-v9 packages in the
repo
@codesandbox-ci
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.

Latest deployment of this branch, based on commit e30e965:

Sandbox Source
Fluent UI React Starter Configuration

@size-auditor
Copy link

size-auditor bot commented Sep 16, 2021

Asset size changes

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

Baseline commit: d05b3db7f100423b47d782ddf457252171395de6 (build)

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-button
Button
22.932 kB
6.984 kB
react-button
CompoundButton
28.215 kB
7.834 kB
react-button
MenuButton
24.733 kB
7.546 kB
react-button
ToggleButton
32.527 kB
7.601 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
163.935 kB
46.761 kB
react-components
react-components: FluentProvider & webLightTheme
35.75 kB
11.392 kB
react-link
Link
13.718 kB
5.66 kB
🤖 This report was generated against d05b3db7f100423b47d782ddf457252171395de6

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react)

No significant results to display.

All results

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

@ling1726 ling1726 merged commit 90d71a0 into microsoft:master Sep 17, 2021
@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

@msft-fluent-ui-bot
Copy link
Collaborator

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

Handy links:

mlp73 pushed a commit to mlp73/fluentui that referenced this pull request Jan 17, 2022
…19840)

Clean up dependency separation between v9 and non-v9 packages in the
repo
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.

5 participants