Skip to content

Conversation

@Antreesy
Copy link
Contributor

Summary

Search input is now reset, when:

  • Navigating within the 'files' view (list)
  • Changing directory (query param of URL and Vue-router)

Before:

2025-10-10_11h40_25.mp4

After:

2025-10-10_11h39_24.mp4

TODO

  • test other cases?

Checklist

@Antreesy Antreesy added this to the Nextcloud 33 milestone Oct 10, 2025
@Antreesy Antreesy self-assigned this Oct 10, 2025
@Antreesy Antreesy requested review from a team and skjnldsv as code owners October 10, 2025 09:44
@Antreesy Antreesy removed the request for review from a team October 10, 2025 09:44
@Antreesy Antreesy added the bug label Oct 10, 2025
@Antreesy Antreesy requested review from sorbaugh and susnux October 10, 2025 09:44
@Antreesy Antreesy added 3. to review Waiting for reviews feature: files labels Oct 10, 2025
@Antreesy Antreesy requested a review from susnux October 10, 2025 14:57
@Antreesy Antreesy force-pushed the fix/53038/clear-search branch from 002485a to 196aa9b Compare October 13, 2025 15:30
@Antreesy
Copy link
Contributor Author

/compile

Signed-off-by: nextcloud-command <[email protected]>
@Antreesy Antreesy merged commit 9c8b500 into master Oct 14, 2025
137 of 139 checks passed
@Antreesy Antreesy deleted the fix/53038/clear-search branch October 14, 2025 07:22
@Antreesy
Copy link
Contributor Author

/backport 196aa9b to stable32

@Antreesy
Copy link
Contributor Author

/backport 196aa9b to stable31

@backportbot

This comment was marked as outdated.

@susnux
Copy link
Contributor

susnux commented Oct 21, 2025

Funny: We have a Cypress test to ensure the old behavior - somehow it did not trigger here...

@susnux
Copy link
Contributor

susnux commented Oct 21, 2025

Test fix: #55892

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clear Search Input on Folder Navigation

5 participants