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

Close the stream when when the background is clicked. #22

Closed
wants to merge 2 commits into from

Conversation

tomlister
Copy link

Hey mate.

Haven't tested this, as I used GitHub's latest and greatest in-browser coding interface (press . on any repository).
Give me a shout if it works or if it doesn't.

Cheers.

@king-millez
Copy link
Member

Awesome. Can you quickly merge the latest commit I did on overhaul-1 into your branch, then I'll test.

@king-millez
Copy link
Member

So it works - sort of.

The VideoJS container sits on top of the background and extends for most of the width of the screen:
image

Clicking below and above it does work though!

@tomlister
Copy link
Author

Could you reduce the width or no?

@king-millez
Copy link
Member

Possibly, the width is assigned with VideoJS, so it could be overridden by some CSS I assume. Do you wanna try it?

@tomlister
Copy link
Author

tomlister commented Aug 13, 2021 via email

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.

2 participants