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

TwicVideo shouldPlay, isLooping and isMuted #107

Open
obibring opened this issue Jul 14, 2024 · 2 comments
Open

TwicVideo shouldPlay, isLooping and isMuted #107

obibring opened this issue Jul 14, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@obibring
Copy link

obibring commented Jul 14, 2024

I'm evaluating TwicPics for my needs and noticed that TwicVideo in react-native hardcodes isLooping, isMuted, and shouldPlay to true. Are there plans to make this configurable? This seems like an important feature for react-native users.

@obibring obibring changed the title TwicVideo isLooping and isMuted TwicVideo shouldPlay, isLooping and isMuted Jul 14, 2024
@mbgspcii
Copy link
Collaborator

Hi @obibring
I understand your concern. That's why I want to clarify that TwicVideo is not a "video player."
It is a component that allows easy integration of videos optimised by TwicPics, which are short, muted, and auto-playing videos. They enable the integration of hero, product, and thumbnail videos.
If you are interested in, this is an article about short videos.

That said, you are right, we could consider disabling autoplay as well as looping. Unfortunately, we are not yet ready to distribute optimised videos with sound.

@mbgspcii mbgspcii added the enhancement New feature or request label Jul 15, 2024
@mbgspcii mbgspcii reopened this Jul 15, 2024
@obibring
Copy link
Author

Hi @mbgspcii, thanks for clarifying. As a developer evaluating TwicPics for my needs, it is surprising that:

  1. Video support on TwicPics is really just animated image support.
  2. I have to upgrade my plan in order to realize that TwicPics does not need my needs - because videos aren't supported in the free plan. Unless I was lucky enough to read the fine-print you linked to (I read the docs extensively and only realized this requirement now, although I do remember that videos could not be longer than 30s)

The word "video" has implications for customers, and customers will be surprised to learn that all videos must loop and must be muted. It would be helpful for customers if TwicPics referred to videos as animated images - or a similar term - to align their expectations with what TwicPics offers.

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

No branches or pull requests

2 participants