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(files): File type filter UI sync with filter state #49259

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Nov 13, 2024

Summary

When changing the folder the filter will be re-mounted by the file list, so we need to pass the current state of the filter to the filter UI.

Checklist

@susnux susnux added this to the Nextcloud 31 milestone Nov 13, 2024
@susnux susnux requested review from a team and removed request for a team November 13, 2024 16:02
@susnux susnux requested a review from skjnldsv as a code owner November 13, 2024 16:02
@susnux susnux self-assigned this Nov 13, 2024
@susnux
Copy link
Contributor Author

susnux commented Nov 13, 2024

/backport to stable30

@susnux
Copy link
Contributor Author

susnux commented Nov 13, 2024

/compile

@susnux susnux force-pushed the fix/file-type-filter-state branch from 67ed7d5 to 7be3fd3 Compare November 17, 2024 00:42
@susnux susnux requested a review from Pytal November 17, 2024 00:42
@blizzz blizzz mentioned this pull request Jan 8, 2025
This was referenced Jan 14, 2025
When changing the folder the filter will be re-mounted by the file list,
so we need to pass the current state of the filter to the filter UI.

Signed-off-by: Ferdinand Thiessen <[email protected]>
Signed-off-by: Ferdinand Thiessen <[email protected]>
@susnux susnux force-pushed the fix/file-type-filter-state branch from 7be3fd3 to e1f10cb Compare January 17, 2025 08:01
@susnux susnux requested a review from Pytal January 17, 2025 08:09
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.

Filter does not survive the change to a subdirectory
3 participants