Skip to content
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

Tenor Implementation Master Issue #13803

Closed
14 of 15 tasks
ScoutHarris opened this issue Apr 2, 2020 · 5 comments · Fixed by #14513
Closed
14 of 15 tasks

Tenor Implementation Master Issue #13803

ScoutHarris opened this issue Apr 2, 2020 · 5 comments · Fixed by #14513

Comments

@ScoutHarris
Copy link
Contributor

ScoutHarris commented Apr 2, 2020

Implement Tenor to replace Giphy for third-party gif import.

Refs:

Tasks:

@frosty
Copy link
Contributor

frosty commented May 11, 2020

Some remaining issues I observed while testing the main PR:

  • Issue in the block editor where I saw search results appear briefly and then vanish. For some reason it happens for me every time with the search term 'blathers'.
  • There's no animation or indication of a gif in the block editor – the classic editor shows a GIF badge in the top left of an image.
  • Editing an image of course breaks the gif and turns it into a static image (it actually didn't even correctly replace the image for me in the classic editor when I tested it). I think we should probably disable image editing for gifs, or at the very least show a prompt when tapped saying that the image won't be animated any more.
    • We need to update to use the new media picker cell overlay prepareForReuse functionality to clear out the overlay contents between each use.

@ScoutHarris
Copy link
Contributor Author

Hey @frosty . Regarding this issue:

There's no animation or indication of a gif in the block editor

I'm seeing the GIFs animate in the block editor. Is there a step I'm missing or something? I simply started a new post with the block editor and added a gif to an image block.

block_editor

@ScoutHarris
Copy link
Contributor Author

Hey @frosty .

We need to update to use the new media picker cell overlay prepareForReuse functionality to clear out the overlay contents between each use.

I'm not quite getting this one. Can you elaborate please?

@frosty
Copy link
Contributor

frosty commented Jun 9, 2020

I'm seeing the GIFs animate in the block editor.

Interesting! I just followed the same steps as you, but it doesn't animate 🤔 I've tried it with several, with the latest develop build, and they're all just static.

@frosty
Copy link
Contributor

frosty commented Jun 9, 2020

I'm not quite getting this one. Can you elaborate please?

@SergioEstevao recently added a prepareForReuse method to ReusableOverlayView: https://github.com/wordpress-mobile/MediaPicker-iOS/pull/351/files

We should implement that in TenorPicker.swift to clear out the overlay views (CachedAnimatedImageView) before they're reused, to ensure they don't show old content.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants