Skip to content

fix: v9 Combobox option focus on open updates#26173

Merged
smhigley merged 3 commits intomicrosoft:masterfrom
smhigley:combo-option-focus
Jan 6, 2023
Merged

fix: v9 Combobox option focus on open updates#26173
smhigley merged 3 commits intomicrosoft:masterfrom
smhigley:combo-option-focus

Conversation

@smhigley
Copy link
Contributor

@smhigley smhigley commented Jan 5, 2023

Previous Behavior

  • Multiselect Combobox & Dropdown began option focus at last selected option when opened (similar to single-select combobox)
  • When opened from the combobox chevron, the option keyboard focus outline still showed up

New Behavior

  • Multiselect combo & dropdown always begin from the first option when opened (unlike single-select) -- discussed with design
  • Focus visibility is reset when using the mouse to open a combobox from the chevron icon

Related Issue(s)

Addresses poitns 1 & 2 in #26069

  • Fixes #

@smhigley smhigley self-assigned this Jan 5, 2023
@smhigley smhigley requested a review from a team as a code owner January 5, 2023 02:46
@github-actions github-actions bot added this to the January Project Cycle Q1 2023 milestone Jan 5, 2023
@size-auditor
Copy link

size-auditor bot commented Jan 5, 2023

Asset size changes

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

Baseline commit: 6daf6e3e4d5f888e77181ecf4f7dd63763d2752d (build)

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-combobox
Combobox (including child components)
79.944 kB
25.571 kB
79.945 kB
25.57 kB
1 B
-1 B
react-combobox
ComboboxField
77.14 kB
25.536 kB
77.141 kB
25.532 kB
1 B
-4 B
react-combobox
Dropdown (including child components)
79.184 kB
25.502 kB
79.179 kB
25.499 kB
-5 B
-3 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
59.888 kB
16.652 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
188.87 kB
53.066 kB
react-components
react-components: FluentProvider & webLightTheme
34.379 kB
11.322 kB
react-portal-compat
PortalCompatProvider
6.069 kB
2.053 kB
🤖 This report was generated against 6daf6e3e4d5f888e77181ecf4f7dd63763d2752d

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 5, 2023

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

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1590 1539 5000
Button mount 1118 1104 5000
FluentProvider mount 1879 1884 5000
FluentProviderWithTheme mount 737 746 10
FluentProviderWithTheme virtual-rerender 696 681 10
FluentProviderWithTheme virtual-rerender-with-unmount 743 735 10
MakeStyles mount 2246 2254 50000
Persona mount 3470 3496 5000
SpinButton mount 2947 2944 5000

@fabricteam
Copy link
Collaborator

🕵 fluentuiv9 No visual regressions between this PR and main

@micahgodbolt
Copy link
Member

confirmed old and new behavior.

@smhigley smhigley merged commit 54b6220 into microsoft:master Jan 6, 2023
q1b pushed a commit to q1b/fluentui that referenced this pull request Jan 24, 2023
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
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.

3 participants