search

TwicPics / components

A Web component library that brings the power of TwicPics to your favorite web framework.
MIT License
54 stars 2 forks source link

TwicVideo `shouldPlay`, `isLooping` and `isMuted` #107

Open obibring opened 4 months ago

obibring commented 4 months ago

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.

mbgspcii commented 4 months ago

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.

obibring commented 4 months ago

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.