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

fix: react-combobox listbox popup width matches trigger width #24733

Merged
merged 3 commits into from
Sep 12, 2022

Conversation

smhigley
Copy link
Contributor

@smhigley smhigley commented Sep 8, 2022

Resolves an item on the design review, #23926

Calculates the trigger width when the popup is opened, and sets a width style to the listbox slot.

@smhigley smhigley self-assigned this Sep 8, 2022
@smhigley smhigley requested a review from a team as a code owner September 8, 2022 23:23
@github-actions github-actions bot added this to the July Project Cycle Q3 2022 milestone Sep 8, 2022
@fabricteam
Copy link
Collaborator

fabricteam commented Sep 8, 2022

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-combobox
Combobox (including child components)
73.863 kB
24.018 kB
74.067 kB
24.078 kB
204 B
60 B
react-combobox
Dropdown (including child components)
73.45 kB
23.927 kB
73.661 kB
23.994 kB
211 B
67 B
Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
187.656 kB
51.96 kB
react-components
react-components: FluentProvider & webLightTheme
33.359 kB
11.004 kB
react-portal-compat
PortalCompatProvider
5.851 kB
1.964 kB
🤖 This report was generated against 7f8bad094dadcdb671d5746848e96b5bd4b5791a

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 8, 2022

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

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

@size-auditor
Copy link

size-auditor bot commented Sep 8, 2022

Asset size changes

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

Baseline commit: 7f8bad094dadcdb671d5746848e96b5bd4b5791a (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 8, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1509 1531 5000
Button mount 1145 1140 5000
FluentProvider mount 1866 1866 5000
FluentProviderWithTheme mount 737 736 10
FluentProviderWithTheme virtual-rerender 706 705 10
FluentProviderWithTheme virtual-rerender-with-unmount 743 747 10
MakeStyles mount 2231 2249 50000
SpinButton mount 3010 2977 5000

@smhigley smhigley merged commit 83b033d into microsoft:master Sep 12, 2022
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Sep 14, 2022
* master: (28 commits)
  Fix value font-weight inside heatmap chart (microsoft#24726)
  Fix legend overflow-indication-text role (microsoft#24756)
  Support custom locale in date axis  (microsoft#24753)
  Cleanup env variables (microsoft#24739)
  ci(github): add GH Action to add issue labels based on new GH issue template (microsoft#24788)
  Update disallowedChangeTypes for newly created packages, to allow only 'prerelease' change types by default (microsoft#24763)
  feat(react-components): Adding missing AvatarGroup exports (microsoft#24770)
  remove unnecessary nohoist (microsoft#24760)
  feat(react-dialog): supports 1st rule of ARIA (microsoft#24525)
  BREAKING: TableCell layouts are handled by layout components (microsoft#24762)
  feat: Implement table cell layout components (microsoft#24773)
  applying package updates
  fix: remove readonly from DetailsList (microsoft#24615)
  chore: Cleaning up tokens in Button components so they better adhere to the design spec (microsoft#24732)
  fix: react-combobox listbox popup width matches trigger width (microsoft#24733)
  fix: react-combobox Option focus outline only shows with keyboard nav (microsoft#24700)
  feat: Publish react-field package, and export from react-components/unstable (microsoft#24235)
  fix: Replacing bottom border styles with text decoration underline in Link (microsoft#24734)
  docs(react-theme): Update readme (microsoft#24755)
  Add tests for hover states (microsoft#24390)
  ...
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