Skip to content
Snippets Groups Projects

Fix: filters bar appearing when there are no filters

Merged Warren Gifford requested to merge fa/filters-empty into master

Created by: attfarhan

Fixes https://github.com/sourcegraph/sourcegraph/issues/1764.

To fix the issue, I simplified how filters contributed from extensions are rendered by requesting Contributions in the SearchResults component and removing the SearchFiltersContainer component. The empty prop that SearchFiltersContainer had did not serve its purpose, as we cannot check whether a child component renders null from a parent component. Also, we never passed in the scope prop, so we didn't need to observe scopeChanges, which made the separate component unnecessary. LMK if I missed something @vanesa @slimsag.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading