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

Prop alwaysShowControls #55

Merged

Conversation

leadmrn
Copy link
Contributor

@leadmrn leadmrn commented Feb 23, 2023

Hi,

I created a new prop which allows to display the controls for the entire duration of the media included. It can be useful for audio media that has no visual content for example.

It's up to you whether you want to include it or not!

@LunatiqueCoder
Copy link
Owner

Hey!

I realy liked everything you came up with so far - I'm sure I'll like this one as well.

Do you need help resolving conflicts?

if you need any help contributing, this comment might help: #49 (comment)

@LunatiqueCoder
Copy link
Owner

Hey @leadmrn , please be sure to also check #49 (comment) , might be really helpful for you!

@leadmrn
Copy link
Contributor Author

leadmrn commented Feb 23, 2023

Ok that's good normally!

@LunatiqueCoder
Copy link
Owner

LunatiqueCoder commented Feb 23, 2023

Okay, @leadmrn, looks good, I'll test them this weekend and will definitely have this! Thank you so much! Really appreciate it!

About the bugs, it's worth mentioning that some of them are from react-native-video. They are working on version 6 and they have a release candidate. I'm personally using that version since it fixes a lot of bugs.

You can try the alpha version using yarn add react-native-video@alpha or npm install --save react-native-video@alpha

@leadmrn
Copy link
Contributor Author

leadmrn commented Feb 24, 2023

No worries, the pleasure is mine.

For the bugs I had already installed react-native-video v6 and it didn't fix the problems :/

@LunatiqueCoder
Copy link
Owner

All right, good to know. I'll check it out this weekend and have it merged. Again, thanks a lot!

@LunatiqueCoder LunatiqueCoder added the enhancement New feature or request label Mar 5, 2023
@LunatiqueCoder
Copy link
Owner

LunatiqueCoder commented Mar 6, 2023

Hey @leadmrn

All right, I checked this one and I think it's very good. One thing I'm not sure about is the name. I'm thinking about something like disableControlsToggle and also mention in the README file that it should be used with showOnStart={true/false} to force the controls into a shown or hidden state. What do you think?

@LunatiqueCoder LunatiqueCoder merged commit 283c83c into LunatiqueCoder:master Sep 21, 2023
4 checks passed
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

Successfully merging this pull request may close these issues.

showOnStart doesn't prevent controls to show when the video starts
2 participants