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.

Sign up for GitHub

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

Jump to bottom

[Dropdown] clearable dropdown should not mutate the open state #3114

Closed
mvorisek opened this issue Oct 26, 2024 · 1 comment
Closed

[Dropdown] clearable dropdown should not mutate the open state #3114

mvorisek opened this issue Oct 26, 2024 · 1 comment
Assignees
@lubber-de
Labels
type/bug Any issue which is a bug or PR which fixes a bug
Milestone
2.9.4

Comments

@mvorisek
Copy link
Contributor

Bug Report

Steps to reproduce

  1. Have a dropdown with multiple options allowed with "clearable" enabled.
  2. Open it.
  3. Click clear icon.
  4. Notice, the dropdown is closed. It should be kept open.

Expected result

Dropdown is kept open.

Actual result

Dropdown is collapsed.
@mvorisek mvorisek added state/awaiting-investigation Anything which needs more investigation state/awaiting-triage Any issues or pull requests which haven't yet been triaged type/bug Any issue which is a bug or PR which fixes a bug labels Oct 26, 2024
@lubber-de lubber-de removed the state/awaiting-triage Any issues or pull requests which haven't yet been triaged label Oct 26, 2024
@lubber-de lubber-de self-assigned this Oct 26, 2024
@lubber-de lubber-de removed the state/awaiting-investigation Anything which needs more investigation label Oct 26, 2024
@lubber-de lubber-de added this to the 2.9.4 milestone Oct 26, 2024
@lubber-de lubber-de mentioned this issue Oct 26, 2024
Merged
@lubber-de
Copy link
Member

lubber-de commented Oct 26, 2024 •
edited
Loading

Implemented/ Fixed by #3115
I think many people still like/want to make a possible open dropdown menu collapse automatically, so i made it an option instead of strictly changing the behavior.

@lubber-de lubber-de added state/has-pr An issue which has a related PR open tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build and removed state/has-pr An issue which has a related PR open labels Oct 26, 2024
@lubber-de lubber-de removed the tag/next-release/nightly Any issue which has a corresponding PR which has been merged and is available in the nightly build label Feb 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lubber-de lubber-de

Labels
type/bug Any issue which is a bug or PR which fixes a bug
Projects
None yet
Milestone
2.9.4
Development

No branches or pull requests
2 participants
@mvorisek @lubber-de