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] Player auto-rotate #4140

Closed
black-cerberon opened this issue Aug 16, 2020 · 11 comments
Closed

[Feature Request] Player auto-rotate #4140

black-cerberon opened this issue Aug 16, 2020 · 11 comments
Labels
feature request Issue is related to a feature in the app

Comments

@black-cerberon
Copy link

Please add forced auto-rotate in video player.

@black-cerberon black-cerberon added the feature request Issue is related to a feature in the app label Aug 16, 2020
@test2a
Copy link

test2a commented Aug 16, 2020

I would love to have this apply for vertical videos. Changing the orientation in Newpipe makes it default and watching a regular video later I have to turn it back on.

@opusforlife2
Copy link
Collaborator

Try this https://github.com/TeamNewPipe/NewPipe/files/5079507/app-debug.zip and reply if you still need forced autorotate.

@black-cerberon
Copy link
Author

@opusforlife2 that's work but better rotate fullscreen media by device rotation.

@opusforlife2
Copy link
Collaborator

Can you rephrase that? I didn't get it.

@black-cerberon
Copy link
Author

Fullscreen media can use system rotation setting, sensor state or bind only one orientation like you implemented right now. Usally many apps read system rotation setting for change screen orientation. When use sensor state app can ignore system setting and rotate everytime if device was rotated. For example you can look at Simple Gallery App - https://github.com/SimpleMobileTools/Simple-Gallery

@opusforlife2
Copy link
Collaborator

I'm asking why do you need sensor based rotation? In the apk I linked above, when you tap on the fullscreen button, if you're watching a vertical video, the video stays in portrait mode, and if you're watching a normal video, then the video rotates to landscape mode.

Where is the need to use sensor state?

@black-cerberon
Copy link
Author

Sorry, I not tested it with vertical videos.

@opusforlife2
Copy link
Collaborator

It's alright. I was just looking for clarification because I suspected your issue might be already solved.

There could still be some users who want sensor based rotation separate from system rotation. If that is the case, it would be up to the devs whether or not to implement it. Or maybe a contributor could decide to take it up.

@test2a
Copy link

test2a commented Aug 16, 2020

Try this https://github.com/TeamNewPipe/NewPipe/files/5079507/app-debug.zip and reply if you still need forced autorotate.

@opusforlife2 this version did the trick although it seems the rotate button is no longer there, not that we need it. What does the globe button do ? i clicked on it and the app crashed for me

Exception

  • User Action: ui error
  • Request: App crash, UI failure
  • Content Country: IN
  • Content Language: en-IN
  • App Language: en_IN
  • Service: none
  • Version: 0.19.8
  • OS: Linux Android 7.0 - 24
Crash log

android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=https://www.youtube.com/... flg=0x10000000 pkg=com.zui.resolver }
	at android.app.Instrumentation.checkStartActivityResult(Instrumentation.java:1816)
	at android.app.Instrumentation.execStartActivity(Instrumentation.java:1525)
	at android.app.Activity.startActivityForResult(Activity.java:4265)
	at androidx.fragment.app.FragmentActivity.startActivityForResult(FragmentActivity.java:675)
	at android.app.Activity.startActivityForResult(Activity.java:4224)
	at androidx.fragment.app.FragmentActivity.startActivityForResult(FragmentActivity.java:662)
	at android.app.Activity.startActivity(Activity.java:4548)
	at android.app.Activity.startActivity(Activity.java:4516)
	at org.schabi.newpipe.util.ShareUtils.openUrlInBrowser(ShareUtils.java:37)
	at org.schabi.newpipe.player.VideoPlayerImpl.onOpenInBrowserClicked(VideoPlayerImpl.java:910)
	at org.schabi.newpipe.player.VideoPlayerImpl.onClick(VideoPlayerImpl.java:790)
	at android.view.View.performClick(View.java:5617)
	at android.view.View$PerformClick.run(View.java:22271)
	at android.os.Handler.handleCallback(Handler.java:751)
	at android.os.Handler.dispatchMessage(Handler.java:95)
	at android.os.Looper.loop(Looper.java:154)
	at android.app.ActivityThread.main(ActivityThread.java:6153)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:868)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:758)


@opusforlife2
Copy link
Collaborator

@test2a If you lock device rotation, the full screen button will appear.

The globe button is to Open In Browser. Your error is the same as in #3925. Can you supply the information that Stypox asked for there?

@test2a
Copy link

test2a commented Aug 16, 2020

@opusforlife2 yes. I have not set a default browser on the phone. When clicking on a link anywhere else, it asks to choose the browser to open this link in.
Now that i have set the default browser the error is not appearing. I did write on the link about having a proper error message instead of a crash, lets see

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issue is related to a feature in the app
Projects
None yet
Development

No branches or pull requests

3 participants