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

Dropdown: Improving multiselect dropdown screen reader behavior. #14377

Merged

Conversation

khmakoto
Copy link
Member

@khmakoto khmakoto commented Aug 6, 2020

Pull request checklist

  • Addresses an existing issue: Fixes #0000
  • Include a change request file using $ yarn change

Description of changes

This PR is a port of fixes #14367 and #14369 done in office-ui-fabric-react to @fluentui/react-next.

Focus areas to test

(optional)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Aug 6, 2020

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

Sandbox Source
Fluent UI Button Configuration
microsoft/fluentui: codesandbox-react-template Configuration
microsoft/fluentui: codesandbox-react-next-template Configuration
microsoft/fluentui: codesandbox-react-northstar-template Configuration

@khmakoto
Copy link
Member Author

khmakoto commented Aug 6, 2020

@msft-github-bot merge in 1 minute.

@msft-github-bot
Copy link
Contributor

Hello @khmakoto!

Because you've given me some instructions on how to help merge this pull request, I'll be modifying my merge approach. Here's how I understand your requirements for merging this pull request:

  • I won't merge this pull request until after the UTC date Thu, 06 Aug 2020 01:16:08 GMT, which is in 1 minute

If this doesn't seem right to you, you can tell me to cancel these instructions and use the auto-merge policy that has been configured for this repository. Try telling me "forget everything I just told you".

@size-auditor
Copy link

size-auditor bot commented Aug 6, 2020

Asset size changes

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

Baseline commit: aea3a8be1e98123cf840bc245274b84621635f4c (build)

@msft-github-bot
Copy link
Contributor

Perf Analysis

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 918 914 5000
ButtonNext mount 639 622 5000
Checkbox mount 1645 1635 5000
CheckboxBase mount 1376 1443 5000
CheckboxNext mount 1686 1738 5000
ChoiceGroup mount 5270 5366 5000
ComboBox mount 952 918 1000
CommandBar mount 7481 7549 1000
ContextualMenu mount 12775 12797 1000
DefaultButton mount 1159 1141 5000
DetailsRow mount 3600 3610 5000
DetailsRowFast mount 3765 3850 5000
DetailsRowNoStyles mount 3627 3539 5000
Dialog mount 1518 1529 1000
DocumentCardTitle mount 1793 1785 1000
Dropdown mount 2783 2646 5000
FocusZone mount 1908 1826 5000
IconButton mount 1866 1896 5000
Label mount 364 340 5000
Link mount 486 458 5000
LinkNext mount 523 520 5000
MenuButton mount 1555 1535 5000
Nav mount 3412 3407 1000
Panel mount 1524 1491 1000
Persona mount 849 893 1000
Pivot mount 1455 1433 1000
PivotNext mount 1401 1423 1000
PrimaryButton mount 1281 1296 5000
SearchBox mount 1337 1343 5000
SearchBoxNext mount 1403 1399 5000
Slider mount 1528 1545 5000
SliderNext mount 1977 2057 5000
SpinButton mount 5166 5088 5000
SpinButtonNext mount 5157 5247 5000
Spinner mount 419 438 5000
SplitButton mount 3216 3292 5000
Stack mount 556 538 5000
StackWithIntrinsicChildren mount 2001 2116 5000
StackWithTextChildren mount 5215 5222 5000
TagPicker mount 2844 2734 5000
Text mount 442 444 5000
TextField mount 1464 1412 5000
ThemeProvider mount 2979 3006 5000
ThemeProvider virtual-rerender 517 508 5000
Toggle mount 868 854 5000
ToggleNext mount 874 833 5000
button mount 114 107 5000

Perf Analysis (Fluent)

⚠️ 1 potential perf regressions detected

Potential regressions comparing to master

Scenario Current PR Ticks Baseline Ticks Ratio Regression Analysis
HeaderMinimalPerf.default 391 385 1.02:1 analysis
Perf comparison
Status Scenario Fluent TPI Fabric TPI Ratio Iterations Ticks
🎯 Avatar.Fluent 0.46 0.48 0.96:1 2000 914
🦄 Button.Fluent 0.12 0.19 0.63:1 5000 584
🔧 Checkbox.Fluent 0.65 0.35 1.86:1 1000 647
🎯 Dialog.Fluent 0.16 0.22 0.73:1 5000 792
🔧 Dropdown.Fluent 2.93 0.47 6.23:1 1000 2934
🔧 Icon.Fluent 0.15 0.05 3:1 5000 745
🦄 Image.Fluent 0.08 0.13 0.62:1 5000 412
🔧 Slider.Fluent 1.58 0.37 4.27:1 1000 1581
🔧 Text.Fluent 0.07 0.03 2.33:1 5000 350
🦄 Tooltip.Fluent 0.1 16.2 0.01:1 5000 509

🔧 Needs work     🎯 On target     🦄 Amazing

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
Image.Fluent 412 364 1.13:1
DividerMinimalPerf.default 408 363 1.12:1
ButtonSlotsPerf.default 673 606 1.11:1
ButtonMinimalPerf.default 186 172 1.08:1
BoxMinimalPerf.default 382 357 1.07:1
CarouselMinimalPerf.default 501 470 1.07:1
PortalMinimalPerf.default 120 112 1.07:1
ReactionMinimalPerf.default 415 387 1.07:1
SegmentMinimalPerf.default 360 336 1.07:1
IconMinimalPerf.default 729 679 1.07:1
ChatMinimalPerf.default 649 614 1.06:1
HierarchicalTreeMinimalPerf.default 461 440 1.05:1
LayoutMinimalPerf.default 410 390 1.05:1
AnimationMinimalPerf.default 407 391 1.04:1
PopupMinimalPerf.default 691 666 1.04:1
StatusMinimalPerf.default 737 708 1.04:1
TextAreaMinimalPerf.default 513 495 1.04:1
Checkbox.Fluent 647 621 1.04:1
FlexMinimalPerf.default 292 284 1.03:1
HeaderSlotsPerf.default 817 797 1.03:1
AlertMinimalPerf.default 326 320 1.02:1
CheckboxMinimalPerf.default 2859 2816 1.02:1
GridMinimalPerf.default 350 342 1.02:1
ListMinimalPerf.default 503 493 1.02:1
ProviderMinimalPerf.default 934 916 1.02:1
RadioGroupMinimalPerf.default 436 427 1.02:1
SliderMinimalPerf.default 1612 1580 1.02:1
TextMinimalPerf.default 358 350 1.02:1
TreeMinimalPerf.default 938 917 1.02:1
TreeWith60ListItems.default 226 221 1.02:1
CardMinimalPerf.default 586 582 1.01:1
ChatDuplicateMessagesPerf.default 427 424 1.01:1
DialogMinimalPerf.default 791 785 1.01:1
InputMinimalPerf.default 1325 1308 1.01:1
LoaderMinimalPerf.default 749 743 1.01:1
MenuMinimalPerf.default 872 867 1.01:1
SplitButtonMinimalPerf.default 3926 3900 1.01:1
TableManyItemsPerf.default 2369 2336 1.01:1
CustomToolbarPrototype.default 3775 3727 1.01:1
TooltipMinimalPerf.default 790 780 1.01:1
VideoMinimalPerf.default 654 648 1.01:1
Avatar.Fluent 914 907 1.01:1
Button.Fluent 584 581 1.01:1
Dialog.Fluent 792 784 1.01:1
Dropdown.Fluent 2934 2910 1.01:1
Icon.Fluent 745 740 1.01:1
Text.Fluent 350 346 1.01:1
AvatarMinimalPerf.default 492 493 1:1
ChatWithPopoverPerf.default 470 472 1:1
DropdownMinimalPerf.default 2855 2864 1:1
FormMinimalPerf.default 431 431 1:1
ItemLayoutMinimalPerf.default 1296 1299 1:1
LabelMinimalPerf.default 415 415 1:1
ListCommonPerf.default 980 983 1:1
ListNestedPerf.default 910 906 1:1
TableMinimalPerf.default 419 421 1:1
AttachmentSlotsPerf.default 1151 1164 0.99:1
ImageMinimalPerf.default 377 380 0.99:1
MenuButtonMinimalPerf.default 1618 1630 0.99:1
ProviderMergeThemesPerf.default 1842 1862 0.99:1
RefMinimalPerf.default 201 203 0.99:1
AccordionMinimalPerf.default 145 148 0.98:1
AttachmentMinimalPerf.default 161 164 0.98:1
ListWith60ListItems.default 1100 1119 0.98:1
ToolbarMinimalPerf.default 961 982 0.98:1
Slider.Fluent 1581 1611 0.98:1
Tooltip.Fluent 509 527 0.97:1
DropdownManyItemsPerf.default 758 797 0.95:1
EmbedMinimalPerf.default 1985 2103 0.94:1

@msft-github-bot msft-github-bot merged commit 0c8f578 into microsoft:master Aug 6, 2020
@khmakoto khmakoto deleted the multiSelectDropdownAccessibility branch August 6, 2020 23:06
tmaster628 pushed a commit to tmaster628/fluentui that referenced this pull request Aug 12, 2020
…rosoft#14377)

#### Pull request checklist

- [ ] Addresses an existing issue: Fixes #0000
- [x] Include a change request file using `$ yarn change`

#### Description of changes

This PR is a port of fixes microsoft#14367 and microsoft#14369 done in `office-ui-fabric-react` to `@fluentui/react-next`.

#### Focus areas to test

(optional)
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.

3 participants