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: opened popup should be dismissed when window resized in select #5629

Closed
rajsite opened this issue Feb 21, 2022 · 0 comments · Fixed by #6452
Closed

fix: opened popup should be dismissed when window resized in select #5629

rajsite opened this issue Feb 21, 2022 · 0 comments · Fixed by #6452
Labels
area:fast-foundation Pertains to fast-foundation improvement A non-feature-adding improvement status:under-consideration Issue is being reviewed by the team.

Comments

@rajsite
Copy link

rajsite commented Feb 21, 2022

🐛 Bug Report

When the window is resized an open pop-up should be dismissed. For example, the pop-up for a select control should close similar to a browser native select control. Another potential option (that doesn't seem to align with browser behavior) is to resize the pop-up so scrollbars are not added to the page:

browservsnativeselect

💻 Repro or Code Sample

See above example. Stackblitz used: https://stackblitz.com/edit/typescript-geugdq

🤔 Expected Behavior

Align with native select and dismiss popup when browser window is resized

😯 Current Behavior

The popup does not respond to the window resize

💁 Possible Solution

  • Listen for browser resize
  • Potentially for some other event, does the native select dismiss for any blur?

🔦 Context

A select that is opened with many different many items causes the page to add scrollbars if the window is resized.

A particularly uncomfortable behavior this causes is double scrollbars for long lists of items after a window resize:
doublescrollbar

🌍 Your Environment

@rajsite rajsite added the status:triage New Issue - needs triage label Feb 21, 2022
@EisenbergEffect EisenbergEffect added area:fast-components area:fast-foundation Pertains to fast-foundation improvement A non-feature-adding improvement status:under-consideration Issue is being reviewed by the team. and removed status:triage New Issue - needs triage labels Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:fast-foundation Pertains to fast-foundation improvement A non-feature-adding improvement status:under-consideration Issue is being reviewed by the team.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants