Skip to content

Add volume control && volume bar && duration display#82

Open
AaronTYZ wants to merge 5 commits intoTheWidlarzGroup:masterfrom
AaronTYZ:master
Open

Add volume control && volume bar && duration display#82
AaronTYZ wants to merge 5 commits intoTheWidlarzGroup:masterfrom
AaronTYZ:master

Conversation

@AaronTYZ
Copy link
Copy Markdown

@AaronTYZ AaronTYZ commented Mar 19, 2018

Volume control new supported.
Volume bar display when tap.
Video duration display along with control bar.
Good Interative.

Before merged, u can ref to here

@pieter888
Copy link
Copy Markdown

I'm still interested in this PR.
1 Issue I have with it is how you automatically mute all video's, Why would you remove the "muted" prop to have no effect? Also adding a prop to show/hide the duration text would be preferred.

I can totally understand that after 11 months you might not have interest in this PR, but I hope at least the owner @cornedor can look into this PR and make it happen.

}

onVolCtrlGrant(e) {
console.log("Test vol ctrl grant start, pos=", e.nativeEvent.pageY);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd recommend prior to this being pulled in to remove all console.logs.


onVolControl(e) {
let volume = this.state.volume;
const senseFactor = 5/4;
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what does 5/4 mean?

@petrogad
Copy link
Copy Markdown

Ah :) Didn't realize this was 11 months old. Do like the capability that it could add and thank you for the work!

@cornedor did you consider this addition?

@suhasgaji19
Copy link
Copy Markdown

In your formatTime method, i think there should be sec = '0' + sec.toString(); instead of minute = '0' + sec.toString();,

dnnsmnstrr added a commit to hpm/react-native-video-player that referenced this pull request Oct 15, 2019
@moskalakamil
Copy link
Copy Markdown
Collaborator

Hey, thanks for your amazing work! We completely rebuilt this library. If you could resolve the conflicts and add it to the latest codebase, that would be great!

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.

6 participants