Skip to content

fix: quick search filters#12429

Open
hamza221 wants to merge 1 commit intomainfrom
fix/quick-filters
Open

fix: quick search filters#12429
hamza221 wants to merge 1 commit intomainfrom
fix/quick-filters

Conversation

@hamza221
Copy link
Contributor

@hamza221 hamza221 commented Feb 16, 2026 •
edited
Loading

Bugs :

  • Hiding the buttons was quick, which makes the buttons un-clickable most of the time
  • setting this.searchInTo to null breaks the app
  • Toggling logic was reversed for attachments and CurrentUser

Changes:

  • Always show active Button (filter) for quick toggling and knowledge about current state

@hamza221
Copy link
Contributor Author

hamza221 commented Feb 16, 2026

/backport to stable5.7

@hamza221
fix: quick search filters
fd59d5f 
Signed-off-by: Hamza <hamzamahjoubi221@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst ChristophWurst is a code owner

@GretaD GretaD Awaiting requested review from GretaD GretaD is a code owner

@kesselb kesselb Awaiting requested review from kesselb kesselb is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

3. to review bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@hamza221