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

Freshly installed 0.20.2 is no more usable on a TV box #4682

Closed
4 tasks done
bodqhrohro opened this issue Oct 28, 2020 · 8 comments · Fixed by #5459
Closed
4 tasks done

Freshly installed 0.20.2 is no more usable on a TV box #4682

bodqhrohro opened this issue Oct 28, 2020 · 8 comments · Fixed by #5459
Labels
Android TV Issue is related to Android TV bug Issue is related to a bug

Comments

@bodqhrohro
Copy link

Checklist

Steps to reproduce the bug

  1. Install LazyIPTV.
  2. Download/create a playlist with YouTube links.
  3. Open the playlist and open some link in NewPipe.

Actual behaviour

I can't set NewPipe up to open the videos in fullscreen player by default anymore.

Expected behavior

Before the recent YouTube breakage, I had NewPipe set up to open and play YouTube links in fullscreen when opened from LazyIPTV. I don't remember how already. Maybe I opened it in the popup player, enabled the mouse emulation, hit the fullscreen button, and NewPipe remembered that and didn't open the tiny player anymore.

Because of the breakage, I did lots of temporary things: made the settings backup from 0.20.1, wiped the application data several times, then uninstalled NewPipe and installed NewPipe polish (which already had this issue), then I uninstalled NewPipe polish and installed NewPipe 0.20.2. But even after importing the old settings, I can't restore the old behaviour anymore.

I presume this is related to the Unified UI thing. This would be strange though, as the status quo persisted even with the 0.20.1 build from F-Droid (did it have this feature enabled already?) I also can't test the behaviour with old versions, as they don't work with YouTube anymore :(

(Huh, why do I end every paragraph with anymore?)

Screenshots/Screen recordings

Logs

Device info

  • Android version/Custom ROM version: 7
  • Device model: x96 mini
@bodqhrohro bodqhrohro added the bug Issue is related to a bug label Oct 28, 2020
@opusforlife2
Copy link
Collaborator

Nothing changed between 0.20.1 and 0.20.2 except the exact code which was needed to fix Youtube video playback. Please check your setup again.

@bodqhrohro
Copy link
Author

Can you then suggest how do I achieve this behaviour? If I hit the fullscreen button in popup mode, it just opens the video info activity, instead of going to fullscreen directly. If I enable an external video player (which was surely disabled though), NewPipe just fails to start with a toast complaining that no streams were found.

@opusforlife2
Copy link
Collaborator

I don't know your setup, obviously, but from what you describe, maybe instead of the popup player, you enabled autoplay on video details, and used mouse emulation to click the full screen button?

@bodqhrohro
Copy link
Author

Autoplay was definitely disabled, though it makes the behaviour close to the previous one, thanks. But pressing the Back button now exits the fullscreen, rather than quitting NewPipe completely, like it was before.

I made some investigation and found out that the translations contain autoplay_by_calling_app_title and autoplay_by_calling_app_summary keys, which don't seem to be used already in the code though. Also, I can't find any autoplay-related setting neither in the recent settings backup nor in the earlier one made in July. So I still presume that there was some special action for opening links from external apps in fullscreen, which was removed from the action list in some version (possibly accidentally), but still remains in the code as a dead action. This explains pretty well why it worked even after I wiped the application data (the action binding still remained), but got broken when I reinstalled the app completely.

@bodqhrohro
Copy link
Author

Oh, and I recall that I didn't actually have to hit that tiny fullscreen button with the mouse cursor before. When I searched and opened some videos from NewPipe UI, focused the video viewport with D-Pad and pressed OK, it went to fullscreen automatically too rather than starting to play in the tiny viewport.

@bodqhrohro
Copy link
Author

I made some investigation and found out that the translations contain autoplay_by_calling_app_title and autoplay_by_calling_app_summary keys, which don't seem to be used already in the code though.

Nah, bisection has shown that those are remnants of a big rework in cbfe91f; that's surely irrelevant as I started using the app this year only.

@opusforlife2
Copy link
Collaborator

O_o

@bodqhrohro
Copy link
Author

If that helps, the first version I ever installed on this box seems to be 0.19.3, so it got broken somewhere in between.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Android TV Issue is related to Android TV bug Issue is related to a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants