Skip to content

search: factor frontend suggestion components

Warren Gifford requested to merge rvt/factor-server-alert-x into main

Created by: rvantonder

Stacked on https://github.com/sourcegraph/sourcegraph/pull/37633

Want to factor lucky search suggestions into its own component. Decided I don't want to create that in search folder and better organized in a new suggestion folder where DidYouMean component also lives. So now:

  • suggestion folder
    • DidYouMean component
    • LuckySearch suggestion component (factored out, renamed)
    • QuerySuggestion.module.scss (renamed, because shared)

Test plan

Semantics-preserving

App preview:

Check out the client app preview documentation to learn more.

Merge request reports