-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Global Search #624
Global Search #624
Conversation
omdxp
commented
Dec 24, 2024
- Add full flow for searching.
- Make shared component cards for search results and corresponding pages.
- Add global search input component.
- Bug fix
- New feature
- Other
…better data retrieval
…roved clarity and structure
…ype safety and clarity
…ges for better user experience
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested it locally, Looks awesome!
added few comments on the code, and since these [model]-card.tsx
components are re-used by the Search component, let's move them up under src/components/
Looks great otherwise!
…ion cards for consistency
…arch and TopBar components
…proved layout consistency
…rt paths for consistency
…, contributors, and contributions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left a few mostly-optional comments, apart from the bug comment (we should note that down)
web/src/utils/search-modal.ts
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe it makes sense for this to live inside web/src/components/search.tsx
…e state with an empty array
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀🔎 pretty fast on staging already