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

Add possibility to render an overlay component on video #131

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mlazari
Copy link

@mlazari mlazari commented Dec 18, 2020

This change adds a new renderOverlayComponent prop that allows to render something on top of the video, but under controls / play button.

Example:

        <VideoPlayer
          video={{uri}}
          style={styles.video}
          videoWidth={320}
          videoHeight={240}
          renderOverlayComponent={() => (
            <Text
              style={{
                color: 'white',
                position: 'absolute',
                right: 20,
                bottom: 20,
              }}>
              Testing
            </Text>
          )}
        />

video-player

@mlazari mlazari force-pushed the add-overlay-component branch from 38f2914 to 82e871f Compare December 22, 2020 10:51
@mlazari mlazari force-pushed the add-overlay-component branch from 82e871f to 89d3d85 Compare July 14, 2022 12:33
This change adds a new `renderOverlayComponent` prop that allows to render something on top of the video, but under controls / play button.
@mlazari mlazari force-pushed the add-overlay-component branch from 62f03a1 to 05e757f Compare January 17, 2024 14:22
@moskalakamil
Copy link
Member

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

@mlazari
Copy link
Author

mlazari commented Dec 4, 2024

@moskalakamil Thank you, I'll take a look when I have time.

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.

3 participants