Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

svelte: Refactor search results page #59029

Merged
merged 2 commits into from
Dec 15, 2023
Merged

Conversation

fkling
Copy link
Contributor

@fkling fkling commented Dec 15, 2023

My main goal was to add an empty state for the search results page, but I took the opportunity to adopt/test some of Taiyab's ideas for the search results page (left sidebar with type filters, more condense results display). It's only a crude approximation since his work is still in flux. It wasn't the goal to strictly follow the design.

2023-12-15_14-42

sg-sk-search-results-final.mp4

Test plan

Manual testing of the prototype.

My main goal was to add an empty state for the search results page, but
I took the opportunity to adopt/test some of Taiyab's ideas for the
search results page (left sidebar with type filters, more condense
results display). It's only a crude approximation since his work is
sitll in flux.
@fkling fkling requested a review from a team December 15, 2023 13:43
@cla-bot cla-bot bot added the cla-signed label Dec 15, 2023
@fkling fkling self-assigned this Dec 15, 2023
@fkling fkling added the team/code-search Issues owned by the code search team label Dec 15, 2023
@fkling fkling mentioned this pull request Dec 15, 2023
20 tasks
@fkling fkling merged commit 474813e into main Dec 15, 2023
@fkling fkling deleted the fkling/sk/search-results-empty-state branch December 15, 2023 17:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-signed team/code-search Issues owned by the code search team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants