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

Can't go directly to fullscreen when tapping thumbnail #4152

Closed
opusforlife2 opened this issue Aug 17, 2020 · 44 comments · Fixed by #5459
Closed

Can't go directly to fullscreen when tapping thumbnail #4152

opusforlife2 opened this issue Aug 17, 2020 · 44 comments · Fixed by #5459
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)

Comments

@opusforlife2
Copy link
Collaborator

opusforlife2 commented Aug 17, 2020

Version

  • 0.20.0

Regression

The user can no longer go into fullscreen directly upon tapping the thumbnail in the video details page, like in 0.19.8. (for those checking: requires locked system rotation and landscape mode in main player)

Additional Note

MUH FULLSCREEN!

@opusforlife2 opusforlife2 added bug Issue is related to a bug player Issues related to any player (main, popup and background) labels Aug 17, 2020
@B0pol
Copy link
Member

B0pol commented Aug 17, 2020

Was it possible before? I can't reproduce in 0.19.8, please give steps to reproduce.

Btw, isn't it a duplicate of #4070?

@opusforlife2
Copy link
Collaborator Author

I did give STR: "(for those checking: requires locked system rotation and landscape mode in main player)"

Btw, isn't it a duplicate of #4070?

That's the issue I was browsing when I remembered I wanted to open this one. That one is for external links. This one is for the thumbnail in video details.

@AnonymousSeal
Copy link

I think this is a good idea. It feels more natural to open the video upon clicking the thumbnail, than opening a info page from where you can start the video. It feels more natural (maybe just because it's what youtube does).

@opusforlife2
Copy link
Collaborator Author

@EliasKomar This issue is for the thumbnail you see in the video details page, not for thumbnails in video lists like search results, playlists, etc. If you want the video to autoplay upon opening like Youtube does, then that's the default behaviour in the upcoming unified player.

@B0pol Sorry, this comment made me realise I didn't specify where the thumbnail was, which led to your questions. I'll edit the OP.

@Stypox
Copy link
Member

Stypox commented Sep 3, 2020

If the fullscreen opened when tapping the thumbnail, it would be an inconsistent and possibly unexpected behaviour with respect to how the video starts playing with autoplay enabled. But at the same time I think the benefits outcome the cons, so I agree this should be changed. @avently what do you think?

@opusforlife2
Copy link
Collaborator Author

Well, yeah. You can only see the thumbnail when autoplay is off. Otherwise, when autoplay is on, the video is already playing and it's one tap to go full screen. If autoplay is off, it's two taps: one to play the video in portrait mode, and another to go full screen.

@avently
Copy link
Contributor

avently commented Sep 3, 2020

@Stypox

what do you think?

I think it's a bad idea. We already discussed it in the unified PR:

  • making the video automatically rotated to landscape from portrait on play looks ugly
  • inconsistency everywhere.

You probably didn't see how it looks like but I did. I use this for more than 1 year long. I added this "mode" just because theScrabi asked about it in my first PR here.

I'm ok if it will be optional and disabled by default but even in this case I will never write any line of code for this "anti-feature".

Yeah, it may sound unpleasant but it's how I feel like when I remember this ugly transition from portrait to landscape on every play.

I just use autorotation enabled globally and the only thing I need to do is to rotate the phone for viewing on fullscreen.

@opusforlife2
Copy link
Collaborator Author

@avently Are you talking about the transition that exists in 0.19.8? Or was yours different in someway?

@avently
Copy link
Contributor

avently commented Sep 3, 2020

Are you talking about the transition that exists in 0.19.8? Or was yours different in someway?

I'm talking about transition that takes place after clicking on fuillscreen button. If the feature you want will be realized than the same action will be made after a stream starts playing. And this is bad in terms of usability. If you want the rotation happens just press on the fullscreen button. It's ok because you really expect that the screen rotates.

@opusforlife2
Copy link
Collaborator Author

If the feature you want will be realized than the same action will be made after a stream starts playing.

Yes, that's what happens in 0.19.8 and all the releases before it, and it's the behaviour I'm used to.

Since the autoplay option didn't exist before, it wasn't a problem. But now that it does, this should be a setting, dependent on the autoplay one. So if autoplay is on, this setting should be greyed out.

But I'm concerned about this:

inconsistency everywhere

Are you talking about bugs appearing elsewhere because of this code change?

@avently
Copy link
Contributor

avently commented Sep 3, 2020

Are you talking about bugs appearing elsewhere because of this code change?

No, it will not produce any bugs, it's just a couple of lines of code but it doesn't look logical choice to rotate the screen after pressing on thumbnail.

@opusforlife2
Copy link
Collaborator Author

Ah well, it would be behind a toggle, so those who don't want it can avoid it.

@avently
Copy link
Contributor

avently commented Sep 24, 2020

Guys, when do you plan to release 0.20.0? Looks like some PR should be merged and we are done?

@Stypox
Copy link
Member

Stypox commented Sep 24, 2020

@avently this Sunday @TobiGr is planning to publish a RC, and these PRs need to be merged: #3178, #4246, #4254, (#4255, but probably not since there is also your solution, we'll see), #4259, #4272, #4288, #4310

@nikhilCad
Copy link

Please make it an opt in and disable it by default.
Something like:
Default- current behaviour
Landscape- turns autoplay off, opens in landscape

@avently
Copy link
Contributor

avently commented Oct 5, 2020

You are talking about fullscreen after play, it can be easily implemented but the problem I see is when the device is in portrait mode. So what do you expect to happen after play when your device is in portrait and horizontal video started to play?
• device stays in the same orientation but opens on fullscreen
• device switches to landscape and opens in fullscreen?

Automatic action can only be made when you have global autorotation disabled (I.e. locked to portrait or landscape).

And a part of the question, what should happen when you stopped watching the video (I.e. video finished or you pressed back button)?

What do you expect to see when the global autorotation is enabled?
• device should go into fullscreen
• nothing should happen?

The problem is these questions I don't have universal answers for. Because some people would want one behavior, another people want other behavior. From my point of view, the app shouldn't go into fullscreen when the rotation is not appropriate for the video. For example, when the device in portrait (with locked orientation) playing horizontal video in fullscreen should only be possible after automatic rotation to landscape. Same for vertical videos: fullscreen should only be possible when device already rotated or rotated automatically to portrait. The autorotation i'm talking about is already implemented (on phones) and you can test how it works in playlists with mixed horizontal and vertical videos: in locked orientation in fullscreen your phone will be rotated to different orientations appropriate for a video orientation.

@nikhilCad
Copy link

nikhilCad commented Oct 5, 2020 via email

@opusforlife2
Copy link
Collaborator Author

@avently The automatic fullscreening behaviour doesn't need to change from the current one at all. Pretend that the feature is implemented such that after the user taps the thumbnail to play, the app immediately auto-taps the full screen button. That's it. Whatever behaviour the app has currently, can continue to be so.

The full screen button should behave as it does now.
The back button should behave as it does now.

And a part of the question, what should happen when you stopped watching the video (I.e. video finished or you pressed back button)?

Personally, I think the player should remain in fullscreen and wait for the user to tap Back to go back to video details. But this can be tested properly only after the basic auto-full screen behaviour is in place. So it doesn't need to be part of the initial commit/effort. We'll get more knowledge about what looks good or bad when we have an apk to test.

What do you expect to see when the global autorotation is enabled?

Nothing should change there, I think. Because full screen is a function of rotating the phone in that case, it shouldn't change anything in the current behaviour. No extra tap required there like it is in locked orientation.

You are talking about fullscreen after play, it can be easily implemented but the problem I see is when the device is in portrait mode.

🤭 We only need auto full screen for portrait mode. The app automatically goes into full screen if you tap the thumbnail in locked landscape rotation.

@sanityormadness
Copy link

sanityormadness commented Oct 5, 2020

I'm finding this new behaviour incredibly irritating:

First, on my Amazon Fire 5th Gen, even in landscape it doesn't automatically play fullscreen, the video starts playing on the details page.

Second, I never want a video to play on the details page (no, not even on the YT website) - all that does is shrink the video. The only instance I would want anything other than fullscreen video or background audio is splitscreen with another app. Which isn't handled by NewPipe (and already worked fine on my phone anyway), so isn't relevant.

I definitely want an option to toggle the old behaviour (I doubt "always play as if the fullscreen button was hit" would be hard. Automatic autoplay was bad enough, but at least that has a toggle...)

@Stypox

Yeah, it may sound unpleasant but it's how I feel like when I remember this ugly transition from portrait to landscape on every play.

I just use autorotation enabled globally and the only thing I need to do is to rotate the phone for viewing on fullscreen.

It's not "ugly", it's practical. I keep my phone locked in portrait mode for everything except video because I don't want it rotating on me if I'm, say, lying in bed. If I'm reading, I want it in portrait near-100% of the time, and I spend more time reading than watching video. All you're doing by removing this as even a choice is adding "friction" in the form of extra clicks - while the video is playing, to boot.

Removing something which clearly some people prefer, even if the ostensible replacement was bug free (it isn't, it's broken on my Fire which is locked in landscape) is my definition of an "anti-feature".

@johanw666
Copy link

johanw666 commented Oct 6, 2020

I agree fully with @sanityormadness . The use case "watching video in bed" is a large one for me, and also an important reason why I have auto rotate off. The autoplay can fortunately be switched off, but if this behaviour does not become possibe I'll either install 0.19.8 again, or build a fork when I get the Rust stuff to compile in Android Studio.

@muzzol
Copy link

muzzol commented Oct 21, 2020

just want to add that my 3 years old son misses this feature so much and his father is tired of clicking in the little corner to put "PJ Masks" in full screen.

@barsupi
Copy link

barsupi commented Oct 28, 2020

I wonder if I'm the only one that doesn't use his.phone with auto rotate activated. i like it in forced portrait mode

Right now. The full screen button 1st change the phone orientation with no way to rotate it back without close thenapp cuz the button is hidden. And if I press again goes to full screen. Super annoying.

Please bring back the old behavior.
Press play and goes to full screen in landscape mode.

@doodhout
Copy link

Yes exactly, I really don't like that the video starts playing in a small viewport, I'd like it to start playing in fullscreen ALWAYS, so please include a setting to be able to use it like that, because it used to work like that before 0.20.0 and I'm rather fond of it.

@neitsab
Copy link

neitsab commented Nov 2, 2020

I'd like to add in support of this request that it isn't just a matter of one sole more tap:

  • if autoplay is on, it is first one tap to display the on screen controls, then at least one more tap on the fullscreen icon, but much more likely 2 to 4+ because the icon is so small and if you hit the video instead it makes the controls disappear, so then it's rinse and repeat.
  • if autoplay is off (my case), add one compulsory tap to the previous description.

It makes the experience really clumsy and tiring tens of times a day. Before I could reliably know that as soon as I tapped anywhere on the video thumbnail, my phone would go in landscape mode and I could start watching, no fuss. Now I am expecting the process to be clumsy and hard...

I should mention that I also have my phone locked in portrait mode, as the very only time I need it in landscape if when I watch videos, which made the previous behavior so appreciated. I have tried turning autorotate on again and couldn't believed how bothering it was: as soon as I slightly moved or needed a different orientation (in bed etc.) my whole UI (outside of NewPipe) would turn and twist back and forth... I locked back in portrait and don't see myself changing. Some phones definitely have finicky gyro sensors.

I am also perfectly fine with this option having to be toggled/set up on purpose in the Settings, no need to change the new defaults, but at least please offer a way to reinstate what was the default for so long. "Don't break the userspace" as they say in kernel land...

Thank you for your time and efforts in developping NewPipe.

@nikhilCad
Copy link

nikhilCad commented Nov 7, 2020

I feel like I am stupid :
1)Open a video and click the little fullscreen button
2)Tap the video to get the title of the video shown
3) Click and drag the title down to the little miniplayer
4) Close the video using X button
5) Congratulations! You are now locked in landscape. Enjoy your life.
Pushing the back button in a videp breaks ot and takes it back to portrait though.

@opusforlife2
Copy link
Collaborator Author

  1. Close the video using X button

Personally, I think if your device is locked in portrait, step 4 should cause it to rotate back to portrait.

@Littlemac123

This comment has been minimized.

@Littlemac123
Copy link

it would be nice if someone could recompile the GUI back from v0.19.8 in to v0.20.2 and upload the APK

@doodhout
Copy link

@Littlemac123, I REALLY don't think it's that simple, only in utopia are UI and the logic behind the UI so detached and compatible they are fully interchangeable.

@opusforlife2
Copy link
Collaborator Author

#4686 (comment) guys.

@doodhout
Copy link

@opusforlife2 I stand corrected. 😅

@HattinGokbori87
Copy link

HattinGokbori87 commented Nov 19, 2020

Really missing this feature. Couldn't stand this behaviour of the official client on mobile (on Tablets and desktops it makes sense) and switched to NewPipe the moment I came to know about it (apart from privacy related reasons).
I personally can't find much importance of the info page anyway where it needs to be made default land-on page on NewPipe, especially considering you can't like the video and the comments section is also just a vanity function (not undermining work here. I know it's hard constantly battling with Google). All the info page does on a smartphone is playing the video in a tiny screen which is unwatchable unless you have a big-screen phone or on a tablet. I preferred the old behaviour of NewPipe where you can instantly start enjoying the video you want to start watching, and if you're interested in the detailed information of the video or reading some comments you could always switch to portrait mode portrait very easily too.
Current behaviour requires us suitor of the old ways and users of +-5 inch phones to either always go through at least extra 2 clicks (3 clicks if autoplay is turned off) for watching every video. Or turning on the autorotate function and be careful about not leaning on any sides while on bed.
Would like to see the return of this feature (can be an option in the settings).
Thanks to all putting effort in this project.

@opusforlife2
Copy link
Collaborator Author

@mof22 You've already made this same comment on the correct issue (4500), so I'm deleting it here. Please don't double post on different issues out of frustration (apologies if that is not the case). Open source projects progress at the pace the developers set when they have free time.

@TeamNewPipe TeamNewPipe deleted a comment from mof22 Mar 27, 2021
@mof22
Copy link

mof22 commented Mar 27, 2021

@opusforlife2
Sorry about that.
I did not know what the most suitable place was to put the comment.
Feel free to erase what looks inadequate.

@Littlemac123

This comment has been minimized.

@opusforlife2
Copy link
Collaborator Author

@Littlemac123 That has nothing to do with this issue. Please check other issues, or open a new one if you don't find a duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is related to a bug player Issues related to any player (main, popup and background)
Projects
None yet