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

Include GifPicker for chat #8712

Closed

Conversation

iceHtwoO
Copy link

@iceHtwoO iceHtwoO commented May 28, 2022

Closes: element-hq/element-meta#321
Similar PR: #5814
Implementation of a Gif Picker.
It sends the gif directly, an alternative would be to just send the URL and the clients render it. This solution would reduce used storage space and could be extended to support different services but it comes with the problem that when the gif is deleted the client cant display it anymore.

Screenshot 2022-05-28 175632

TODO

  • Testing
  • Darkmode "Powered By GIPHY"
  • BUG: Duplicate gifs break Trending
  • Load more on scroll
  • 2 Columns
  • Settings toggle
  • Defaults to Trending

Signed-off-by: Alexander Neuhäuser [email protected]


Here's what your changelog entry will look like:

✨ Features

@iceHtwoO iceHtwoO requested a review from a team as a code owner May 28, 2022 16:11
@iceHtwoO
Copy link
Author

Hi,
due to me not being a frontend developer I'm not so familiar with ts and react so some feedback on code, design and bugs would be appreciated.

@turt2live
Copy link
Member

Thank you for working on this, however per element-hq/element-meta#321 (comment) I don't think we can safely keep this sort of change open. The feature requires design and product review and is generally quite large in size - we might try to take the work on ourselves if/when product/design get a chance to take a look.

@turt2live turt2live closed this May 28, 2022
@MadLittleMods MadLittleMods added Z-Community-PR Issue is solved by a community member's PR X-Blocked The PR cannot move forward in any capacity until an action is made T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements labels Jun 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements X-Blocked The PR cannot move forward in any capacity until an action is made Z-Community-PR Issue is solved by a community member's PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for a GIF keyboard / ability to easily send GIFs
3 participants