Skip to content

fix: added reset search method - #704

Merged
OwenCoogan merged 1 commit into
masterfrom
oc/DRA-2654
Sep 15, 2026
Merged

OwenCoogan merged 1 commit into
masterfrom
oc/DRA-2654

Conversation

@OwenCoogan

@OwenCoogan OwenCoogan commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

https://www.loom.com/share/5879bdc463d14439ab3ab2085ca7da6f

This small fix clears stale search input state in the OSS power select when the dropdown is closed.

  • resets the search value via the onSearch callback with an empty string
  • prevents old query text from being reused when the component is reopened
  • keeps the list filtering behavior consistent with the current open/closed state

Related to: #

What are the observable changes?

Good PR checklist

  • Title makes sense
  • Is against the correct branch
  • Only addresses one issue
  • Properly assigned
  • Added/updated tests
  • Added/updated documentation with Figma design link. Don't forget to replace "design" by "file" in the URL. For example https://www.figma.com/file/example
  • Migrated touched components to Glimmer Components
  • Properly labeled

@linear-code

linear-code Bot commented Sep 14, 2026

Copy link
Copy Markdown

DRA-2654

@edouardmisset edouardmisset left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job

Copy link
Copy Markdown
Contributor Author

image.png

:trollface:

@OwenCoogan
OwenCoogan merged commit 6369836 into master Sep 15, 2026
5 checks passed
@OwenCoogan
OwenCoogan deleted the oc/DRA-2654 branch September 15, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants