feat: gifs in chat #5159
feat: gifs in chat #5159
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| limit: '20', | ||
| favoritesEnabled: open, | ||
| }); | ||
| const [debounceNextPage] = useDebounceFn<void>(() => { |
There was a problem hiding this comment.
Needed to debounce this because Tenor has a 1RPS rate limit, and without this, whenever we re-fetched after a scroll, it resulted in two calls. In production this could result in hitting rate limits.
| }); | ||
| const [open, setOpen] = useState(false); | ||
| const [query, setQuery] = useState(''); | ||
| const [savedSelection, setSavedSelection] = React.useState<[number, number]>([ |
There was a problem hiding this comment.
What does this mean the number, number?
There was a problem hiding this comment.
Its the cursor / selection the user has made in the input area, where the selected gif will be inserted :)
| side="top" | ||
| align="start" | ||
| avoidCollisions | ||
| className="flex h-[25rem] w-screen flex-col rounded-16 border border-border-subtlest-tertiary bg-background-popover p-4 data-[side=bottom]:mt-1 data-[side=top]:mb-1 tablet:w-[31.25rem]" |
There was a problem hiding this comment.
Have you checked this one on mobile as well?
There was a problem hiding this comment.
yeah, works well on mobile :)
Screen.Recording.2025-12-18.at.12.03.01.mov
| return lastPage.next || undefined; | ||
| }, | ||
| initialPageParam: undefined, | ||
| enabled: !!query, |
There was a problem hiding this comment.
We can also do some throttling here? maybe it's safer (over the debounce)
There was a problem hiding this comment.
What exactly are you thinking of?
The debounce seems to work quite well
There was a problem hiding this comment.
Maybe add some staleTime as I don't think they'll likely change much and the query key includes query already.
| return lastPage.next || undefined; | ||
| }, | ||
| initialPageParam: undefined, | ||
| enabled: !!query, |
There was a problem hiding this comment.
Maybe add some staleTime as I don't think they'll likely change much and the query key includes query already.
Changes
Make chats a little bit more interesting and fun :)
Screen.Recording.2025-12-16.at.12.44.21.mov
Events
Did you introduce any new tracking events?
Experiment
Did you introduce any new experiments?
Manual Testing
Caution
Please make sure existing components are not breaking/affected by this PR
Preview domain
https://gif-integration.preview.app.daily.dev