Skip to content
This repository has been archived by the owner on Oct 7, 2024. It is now read-only.

Transparent background + video example #820

Merged
merged 2 commits into from
Aug 22, 2018
Merged

Conversation

langsmith
Copy link
Contributor

Resolves #802 . Still need to figure out why the VideoView isn't filling the entire screen, despite match_parent for width/height in XML 🤔

ezgif com-resize

@langsmith
Copy link
Contributor Author

langsmith commented Aug 10, 2018

Figured it out. The water now takes up the entire screen in the background.

The video of moving water in the background, makes it hard to record a GIF to post here. Welcome to switch to this branch and install the app on your phone.

screen shot 2018-08-10 at 10 30 04 am

Copy link
Contributor

@osana osana left a comment

Choose a reason for hiding this comment

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

I think we should stop the video in onDestroy

}

@Override
protected void onDestroy() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Given that you start the video onCreate
Should you stop it in onDestroy?
I would be curious to see how it works on a tablet on Orientation change. My tablet is at home in PA though :(

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yea, I should stop it in onDestroy(). Fix made and pushed to this pr.

Still funky when in tablet landscape though 😞

ezgif com-optimize 6

Copy link
Contributor

Choose a reason for hiding this comment

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

@langsmith What happens if you start in landscape? Is it ok then?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Made your suggestions regarding onDestroy()

Creation/destruction seems to be 👌

ezgif com-resize

@langsmith
Copy link
Contributor Author

Ok, @osana . I think this is ready for another review. Things are working with onDestroy() and I decided to lock the example's orientation to portrait 😄

@langsmith langsmith merged commit 04f7048 into master Aug 22, 2018
@langsmith langsmith deleted the ls-transparent-background branch August 22, 2018 17:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants