Skip to content
Snippets Groups Projects

JetBrains: Extract SearchResultHeader and SelectableSearchResult components and clean up CSS

Merged Warren Gifford requested to merge ps/jetbrains-theming-2 into main

Created by: philipp-spiess

Closes #36155 (closed) Closes #36154 (closed) Part-of #36703 (closed)

This PR contains a bunch of cleanups of the search result list React components in preparation of #36703 (closed). This includes:

  • Extraction of a SelectableSearchResult and a SearchResultHeader wrapper component that applies the styling for, well, selectable rows and unelectable (header) rows.
  • Copy SearchResult.module.scss for every sub-type and eliminate unused CSS classes to clean up the styling.

Test plan

The changes were tested locally so there shouldn't be any regressions (but even if, this is just the ground work for more styling related changes soon so it's not a big deal)

Here's an obligatory screenshot but since this is not based on the reference design in Figma, :shrug:

Screenshot 2022-06-07 at 14 28 52

App preview:

Check out the client app preview documentation to learn more.

Merge request reports

Merged by avatar (Apr 12, 2025 11:51pm UTC)

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