Skip to content

chore(react-combobox): move freeform and disabled to ComboboxBase types#30947

Merged
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-combobox/chore--move-freeform-and-disabled-to-ComboboxBase-types
Apr 4, 2024
Merged

chore(react-combobox): move freeform and disabled to ComboboxBase types#30947
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-combobox/chore--move-freeform-and-disabled-to-ComboboxBase-types

Conversation

@bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Apr 3, 2024

Previous Behavior

New Behavior

  1. moves freeform and disabled from ComboboxProps and ComboboxState to ComboboxBaseProps and ComboboxBaseState
  2. moves internal selection logic from useCombobox to useComboboxState
  3. adds freeform support for TagPicker
  4. ensures modifications in @fluentui/react-combobox are followed on @fluentui/react-tag-picker-preview

Related Issue(s)

  • Fixes #

@bsunderhus bsunderhus self-assigned this Apr 3, 2024
@github-actions github-actions bot added this to the April Project Cycle Q1 2024 milestone Apr 3, 2024
@bsunderhus bsunderhus changed the title chore(react-combobox): move freeform and disabled to ComboboxBase types chore(react-combobox): move freeform and disabled to ComboboxBase types Apr 3, 2024
@fabricteam
Copy link
Collaborator

fabricteam commented Apr 3, 2024

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 638 670 5000
Button mount 314 303 5000
Field mount 1170 1152 5000
FluentProvider mount 724 720 5000
FluentProviderWithTheme mount 100 90 10
FluentProviderWithTheme virtual-rerender 82 75 10
FluentProviderWithTheme virtual-rerender-with-unmount 79 95 10
MakeStyles mount 844 849 50000
Persona mount 1754 1703 5000
SpinButton mount 1426 1367 5000

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 3, 2024

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.

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 3, 2024

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-combobox
Combobox (including child components)
102.735 kB
33.197 kB
102.873 kB
33.219 kB
138 B
22 B
react-combobox
Dropdown (including child components)
104.06 kB
33.101 kB
104.339 kB
33.178 kB
279 B
77 B
react-timepicker-compat
TimePicker
104.756 kB
34.581 kB
104.894 kB
34.591 kB
138 B
10 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
71.098 kB
20.515 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
219.521 kB
62.081 kB
react-components
react-components: FluentProvider & webLightTheme
43.585 kB
14.352 kB
react-portal-compat
PortalCompatProvider
7.944 kB
2.588 kB
🤖 This report was generated against fd38bbcbb31e230c19a9c552fa03017945089b9d

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 3, 2024

🕵 fluentuiv9 No visual regressions between this PR and main

@bsunderhus bsunderhus force-pushed the react-combobox/chore--move-freeform-and-disabled-to-ComboboxBase-types branch 2 times, most recently from aea4eb8 to 5fa3819 Compare April 3, 2024 09:50
@bsunderhus bsunderhus force-pushed the react-combobox/chore--move-freeform-and-disabled-to-ComboboxBase-types branch from 5fa3819 to 412fd3e Compare April 3, 2024 11:59
@bsunderhus bsunderhus mentioned this pull request Apr 3, 2024
46 tasks
@bsunderhus bsunderhus marked this pull request as ready for review April 3, 2024 14:13
@bsunderhus bsunderhus requested review from a team and smhigley as code owners April 3, 2024 14:13
@bsunderhus bsunderhus merged commit 8afa361 into microsoft:master Apr 4, 2024
@bsunderhus bsunderhus deleted the react-combobox/chore--move-freeform-and-disabled-to-ComboboxBase-types branch April 4, 2024 07:46
robertpenner pushed a commit to robertpenner/fluentui that referenced this pull request Apr 11, 2024
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