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

Feature request: True FullScreen mode for Android with ExoPlayer #1251

Closed
ionut-movila opened this issue Sep 26, 2018 · 5 comments
Closed

Feature request: True FullScreen mode for Android with ExoPlayer #1251

ionut-movila opened this issue Sep 26, 2018 · 5 comments
Labels
stale Closed due to inactivity or lack or resources

Comments

@ionut-movila
Copy link

Current behavior

The lib. does not support a full screen mode for Android at this moment. Only disabling status bar and buttons bar, with method ReactExoplayerView.setFullscreen but that is far from what a full functional full screen.

Expected behavior

Full screen mode should work in a similar way with what we have on iOS. The Video element should be re-parented to a new native view that will cover existing app content, like a full screen modal. This should also include all common controls: video scrubbing, volume controls, change aspect ratio + exit full screen.

This should work for Android ExoPlayer together with or without option useTextureView.

A good starting point I have found is this answer https://stackoverflow.com/a/48511657.
I say approach no. 1 would work ok, even if it has some compromises for Android < 6 (API 23).
And also using SimpleExoPlayerView (or new PlayerView) for a full screen transition I guess is a good option.

Not to say that we really do need this feature in our app, but I would like to hear other opinions, and why not volunteers to implement it.

@cobarx
Copy link
Contributor

cobarx commented Oct 14, 2018

I'm not sure what we get by re-parenting the video. On iOS, that code is super buggy and doesn't work about half the time. Imo, it's a lot easier to work with to just set the video to have the same bounds as the device and call it a day.

The biggest obstacle is that we don't have a set of controls with all the features you're mentioning. The ExoPlayer sample controls are super basic and don't look good imo.

I have built several iterations of controls as React Native components that I will eventually be able to open source. I'd be curious of your thoughts on whether we should be aiming for native controls or a React Native component is sufficient. If we do the latter, should we also make them available on iOS?

@xstable
Copy link

xstable commented Feb 28, 2019

You might have a look at my repo.
I fork and upgraded react-native-video-player so that it now support react-native-video 4.4.0. Feel free to give it a try.

Because here, @IbrahimSulai is working on a build-in Fullscreen functionality, you can take my solution as temporary workaround

@IbrahimSulai
Copy link
Contributor

Opened a PR (#1730) for full-screen support in Android Exoplayer.

@petrosmm
Copy link

petrosmm commented Apr 9, 2020

@ionut-movila I would just like to add the use of PlayerView, is how I was able to get video supported from android 4.4 and up in my native android application. If you guys added this feature, it would add 4.4 compatibility for sure.

@huynextlevel
Copy link

huynextlevel commented Jul 22, 2021

Hi @IbrahimSulai,
I also added fullscreen={true} for android exoplayer, but It's still not working as expected (Don't see full screen in the control bar)
Screen Shot 2021-07-22 at 13 54 35
Screen Shot 2021-07-22 at 13 55 43

@hueniverse hueniverse added the stale Closed due to inactivity or lack or resources label May 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Closed due to inactivity or lack or resources
Projects
None yet
Development

No branches or pull requests

7 participants