-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Regression - Share and other buttons in video details #4039
Comments
What do you call
? |
Same as I said here: #2907 (comment) Basically, we currently have a Share button in the top bar (menu bar?) which is gone now, since the entire bar is gone after your PR. This contained the resolution selector and the share button. Resolution selector isn't that important because using it shows intent to play the video anyway, so you might as well change it after playing the video. But not having a way to share the link without playing the video first, is a regression. So I suggested to put the Share button here: I thought you didn't like the idea. Now I get the feeling it was just a communication gap between us. 😆 |
I just didn't get in the PR thread what you mean by autoplay toggle:) I didn't even think about that area in the page because it's hidden all the time (comments is visible almost always)
The issue with a share button is a gap between us. I use it a couple of times in my life and I don't think it should be visible on the page at. It's not useless but it's unused all the time. So I think the current position is good for an unused element. Let's say someone will come here and say that he wants to see subtitles, or mute, or scale button near autoplay toggle, what will you say? You'll probably say that it's insane idea because these things are unused. This is how I feelling about share button. |
@avently: I'd like to point out that there is a difference between the in-player share button and the share button that previously was at the top. The latter keeps the original URL, while the later first simplifies the URL and then adds the current timestamp (if possible for that service). I do agree that next to the autoplay button isn't the right location for it imho. |
So is there situation when you need the first instead of the second? If so, it is possible to show an AlertDialog with both links. Isn't it the fix? |
They are in the same situation as the resolution selector. As I said above, you only need these once the video is already playing. But that is not the case with sharing. Having to first play a video before being able to share it doesn't make sense. Here's how it would go:
Now it's either:
or, if you implement an AlertDialog:
Your solution is still the same number of steps, just like before. The only thing it is helping with is that the manual timestamp removal is gone. On the current release, this is the how it goes:
See the difference? As for a counter example, take the resolution selector.
After your PR:
That's it. The only addition there is a few seconds. So I don't mind it much.
Yeah, it's just the first empty space I thought of that didn't seem problematic. Any ideas? |
I'm not saying it's for effective than yours in terms of number of steps. But it's better logically (unused thing on unused place).
I see only only two solutions:
I don't like both solutions but second one is probalby the way how it should work. Don't forget that there is a open in browser button which should be moved out of player too. |
Actually, I'm not sure why that button exists. You can select your browser from the Share sheet directly. |
Hmm. How about a left to right swipeable list where the 4 action buttons are? The default visible actions would be the most used ones, and the lesser used ones could be swiped into view, 4 at a time. Maybe the buttons could change positions based on frequency of use. If this is a viable solution, then I think swiping them to view 4 at a time would be better than a loose scrollable list, because tapping on the action buttons could become tedious. |
That's great solution! Not sure about frequency of use because buttons changing their place is kind of annoying and unexpected. Better would be to allow to rearrange them via long tap and drag. But scrollable list is useful. Not many buttons have to be there. Maybe someone will implement it, I hope so. |
Yeah, that makes more sense. |
Swipe doesn't work with Android TV |
An arrow indicator at the right for TV, then? When you reach the rightmost action button, another right arrow command from the remote would scroll ahead. |
@opusforlife2: I'm pretty sure it'll just go to the next button when you press right at the last visible button on TV in the kind of swiping thing you meant. Edit: I also think it should show as many buttons as possible on that device, e.g. a tablet in landscape will definitely be able to show more than 4 buttons I think. |
Oh. Then what about this? When the cursor (box) is on the rightmost action button, another right command would shift the cursor to the right arrow itself. Then it can be clicked on to do the swipe action. |
@opusforlife2: I meant that your initial swiping idea should just work fine out of the box on Android TV, similar to how you could also e.g. just scroll in the search results (but horizontally instead of vertically obviously). |
Oh. I thought you meant the cursor would shift to the suggested videos list on the right. |
Actually, now that I think about it, that right arrow indicator should be present for all form factors, not just TV. We have to let the user know there is something new there, after all. A right arrow beside the download button would be obvious enough to be noticed. Otherwise mobile users would likely not know about this feature until they accidentally swipe sideways in just the right way or read about the feature online. Moreover, once the user has swiped left once to access the next 4 action buttons, we should also show a left arrow indicator next to the first action button, as well. |
Wow. @XiangRongLin already suggested a scrollable action bar months ago here: #3583 (comment) |
Hello, I'd like to know why the action bar was removed :( My suggestion is that these action buttons are way, way more important than comments or related videos. @opusforlife2 I believe it is now master branch , 0.20.0 |
Changed. Thanks. |
For the record, I use the Share button on the details page all the time after I've finished watching. Know why? #2277 (First level-only comments are pointless on the vast majority of videos, so I don't even show them in NewPipe.) |
No, because popup player has less free room for icon |
@rancidfrog This is off-topic here. Could you open a new issue for this? Thanks. |
@opusforlife2 |
There is no space here @rancidfrog And even if theorically there is space, popup player is supposed to be minimalistic and not bloated. Android's picture in picture mode has only a play / pause button. |
Just to bother you. And, this is the smallest pop-up player can go. For future comments regarding this issue #4601 This might be a slight design change in regards to the implementation, but it is the same logic. |
I suppose many NP users frequently revisit NP for the video they already know or have. They are not interested in watching, but in fetching the link to this particular video to share. The very majority of my shares were not from the time I was watching/listening a video, but from later, when I needed the link reference. If I have to watch video anytime I want its link, it does not make much sense. It is like if I had to download a file from a web site each time I want the link to it. |
@poutnikl |
@rancidfrog It may work for video watching. But for many cases, I am only interested in downloading the video, or just the audio part. Often, when fast connection is available, I do not have time or mood to watch/listen and vice versa. Later I may eventually share the YT link for others to watch/listen. This may be even after long time. I would understand if some devs consider sharing as seldom used function and it's button should not be so exposed, moving it elsewhere. As people have different POV. But putting it exclusively to the player window is semantically just wrong, as this functionality has nothing to do specifically with the player functionality. EDIT: There is the share option if you long press on result list item. For strange reasons I missed the last sharing item at the bottom of the context menu previously. I find the lost sharing from details page more handy, but even this is more useful than going to player. |
One serious drawback for the current share button location is, that you need to start (and also redownload!!) the video again to be able to access the functionality. This is my workflow: When I want to check the comments, I click on the video in my "downloaded" tab, and "show info". Then I need to start the video, but I'm not on my home internet connection anymore rather on my mobile internet connection which is a scarce resource, and I really need to think twice if I want to watch a video on that connection. 1 GiB data usage is not much for a whole month (in my case). I think the share button should be accessible without causing a video (re)download. Sorry for the long explanation and thank you for your time and attention. |
Why is this taking so long to fix? It's super annoying that one of the most used feature was removed. :( |
This comment has been minimized.
This comment has been minimized.
Why do that with the stable branch though? :) |
@Hund There is a PR linked which you can test and give feedback on.
There is no "stable branch". Newpipe has only one release track. @nbmrjuhneibkr I'm minimising your comment because of its passive-aggressiveness. |
Go cast your vote at #4534 (comment) |
Woohoo! My Share button! |
Version
Problem
Now that the unified player PR is merged, let me be the first to add a bug to the Unified Player project (https://github.com/TeamNewPipe/NewPipe/projects/17).
We need to find a way to place the Share button back somewhere on the video details page.
Edit: List of buttons reported to be missing:
List of other possibly useful buttons:
Solution
My suggestion was that it could be placed next to the Autoplay toggle.From the discussion below:
The text was updated successfully, but these errors were encountered: