updates search result cards to match redesign in the feed

[syqs]

https://www.loom.com/share/300b3f49475246feaf15e5feb552144a

Before and after in the loom

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Review	Updated (UTC)
web	Ready	Preview, Comment	Dec 15, 2025 3:36pm

[yattias]

Gave it some thought and I think we should use a context. Reason being, our UI decision making is quite dependent on where its rendered. We could always keep passing props but I'd rather avoid doing so and make UI decisions based on the context.

Example:

type FeedView = 'feed' | 'search' | 'profile' | 'lists';

const FeedViewContext = createContext<FeedView>('feed');

export const useFeedView = () => useContext(FeedViewContext);

Then

<FeedViewContext.Provider value="search">
  <SearchPageContent ... />
</FeedViewContext.Provider>

Then in FeedItemActions

const { feedView } = useFeedView()

const showBookmark = feedView == 'search'

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud