Skip to content

feat: add GIF support check in media block #4710

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Subham-KRLX
Copy link

@Subham-KRLX Subham-KRLX commented Jun 14, 2025

Added logic to accept .gif files as valid media types in the media.js block.

Changes

  • Created js/blocks/media.js
  • Added isSupportedMediaType() that supports images and .gif
  • Test

  • Tested with file objects mimicking GIFs — passed.

Copy link

✅ All Jest tests passed! This PR is ready to merge.

@omsuneri
Copy link
Member

@Subham-KRLX try to add some demos like video or screenshots of the functionality that you achieved by this pr it's very helpful for reviewing!!

@Subham-KRLX
Copy link
Author

Thanks for the quick feedback! I’ll add a demo screenshot or short video showcasing the .gif support in the media block shortly. Let me know if there’s anything specific you’d like to see in the demo!

@Subham-KRLX
Copy link
Author

Hi @omsuneri,
Here's a quick demo screenshot showing .gif support working in the Media Block.
The .gif was successfully uploaded and appears on the stage as expected. Let me know if you'd like a video recording or exported project as well!

![Demo Screenshot]
WhatsApp Image 2025-06-14 at 11 45 20

@Subham-KRLX
Copy link
Author

Hi again 👋
As a follow-up to the earlier screenshot, here's a more detailed view of the .gif being used across multiple media blocks.
It confirms consistent rendering and integration.
Let me know if a video or exported project would also be helpful!
Screenshot 2025-06-14 at 11 47 18 AM

@Subham-KRLX
Copy link
Author

Hi @omsuneri, just checking in! All tests are passing and demo screenshots are added. Let me know if anything else is needed. 🙂

@omsuneri
Copy link
Member

@Subham-KRLX please keep patience will review this soon

@Subham-KRLX
Copy link
Author

@omsuneri Thanks for the update! I’ll keep this PR ready for your review.

Quick note: I’ve prepared a screen recording demo and tested edge cases (large/animated GIFs) if needed.

No rush—just confirming everything’s on standby. Appreciate your time!@omsuneri Thanks for the update! I’ll keep this PR ready for your review.

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

Successfully merging this pull request may close these issues.

2 participants