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(compass-sidebar): fixes the filtering behaviour for databases and collections COMPASS-8026 #5959

Closed
wants to merge 4 commits into from

Conversation

himanshusinghs
Copy link
Contributor

Description

This PR brings back the original filtering behaviour that was bugged in one of the recent changes. Ideally when you are filtering and the filter matches the database then all of the collections of the database must get listed (regardless of collections matching the filter or not).

Checklist

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@paula-stacho
Copy link
Contributor

paula-stacho commented Jun 21, 2024

wait, what does listed mean? we want them included, but not expanded. we've been expanding them previously and this behaviour was removed intentionally (confirmed with Ben)

Copy link
Contributor

@paula-stacho paula-stacho left a comment

Choose a reason for hiding this comment

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

This is not a change request, more of discussion request :) I removed the behaviour intentionally to improve the search for longer lists, where including "non-matches" in the results makes it harder to spot the actual results (they might even be out of sight below all the other items). The user can still expand the filter results, so we need to include the children. But they don't need to clutter the search results.
Sorry if I failed to make that clear before!
I can see how in some scenarios the original approach might be better, so maybe we can discuss this with design again!

@himanshusinghs himanshusinghs marked this pull request as draft June 26, 2024 13:30
@himanshusinghs
Copy link
Contributor Author

Closing this as we will do the refactor entirely

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.

None yet

3 participants