chose: unify shared code between search results components and clean up
Created by: limitedmage
- Move
CodeHostIcon
andSearchResultStar
from their duplicate location inFileSearchResult
,RepoSearchResult
andCommitSearchResult
intoResultContainer
- Remove redundant and unused properties
- Remove
isInContext
fromMatchGroup
, which is written to but never read, as far as I can tell
Recommend reviewing with ignore whitespace enabled
Test plan
Refactor only, all existing tests should pass. Some snapshots were updated but there should be no visual changes.
App preview:
Check out the client app preview documentation to learn more.