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 custom VideoSurface for the VideoPlayer example #3336

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DrewCarlson
Copy link

With this approach the Video can be properly displayed within your Compose UI and removes the need for a SwingPanel.

image

With this approach the Video can be properly displayed within your Compose UI and removes the need for a SwingPanel.
@mahozad
Copy link
Contributor

mahozad commented Jul 13, 2023

I think the following pull request is relevant:

@DrewCarlson
Copy link
Author

Thanks, did not see that one. Would be great to have either merged, the solution is not very discoverable as a PR.

This feels like a bad case to demonstrate Swing interop with. There are better examples and given the limitations there is little value in a video player that cannot have overlapping controls.

@avently
Copy link

avently commented Sep 26, 2023

Hey, @DrewCarlson. You did perfect job, works like a charm! I spend so many hours trying to make Swing variant working but you just give me a chance to remove this problem from my desk. Thank you!

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.

None yet

3 participants