-
-
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
Unified player should be optional, makes the app useless in certain scenarios #4460
Comments
Yes! Pleeaase make it optional! I can only speak for myself, but let me tell you this: Do you, as a dev, really want your users to use your app less than before? |
Hmmm... option to have separate queues for the 3 players? Maybe pick and choose 2 players to have a common queue (like main and popup, but separate background queue)? |
I don't use the popup player, so I can't comment on that, but there definitely needs to be an option to have a separate queue for the background player. Lots of people keep their audio and video players separate from each other. |
I would love that option as well. What you play in the background player is usually very different from what you play in the main or popup players. |
So you think that when you talk like this the devs will want to help you? |
I'm thinking that it's rather strange that the devs let a major regression like this make its way into release after such a long development process. All of the statements you quoted are factually correct in the context of this issue. |
The problem is precisely that "long development process" you mention. Because this was such a massive change, there were bound to be lots of bugs in the final release, even though we tried to spot as many of them as possible. Now what we need is rapid user feedback and some small releases to address critical bugs, and eventually those of lesser and lesser urgency in further releases. |
The problem with this one is that it isn't really a bug, but rather a deliberate design decision that completely removes previously possible usage scenarios due to its limitations. I submitted it as a bug anyway, and described my reasons for doing this. |
I wasn't addressing this bug in particular, but just the general flood of new issues opened after the release was published. We expected this would happen. Now we will work to address them one by one. |
for the seek of playing 360p videos on my replicant os i do give my vote on this! unified player should be optional. |
@panacist Wat. What does that have anything to do with this issue? |
seconding for making it optional, there so many bugs (or weird behaviors) thats make imposible to use |
Oh hey, long time no see, @kapodamy! Why are you not on the IRC channel? |
I would agree, these are not code bugs, but usability bugs = bad UI design and bad UX due to sudden changes in the behaviour. The new version is kind of depressing (UX wise. It seems the app is more stable, crashing less often and a bit snappier). |
Maybe devs wants new fork of entire project? For me, version 0.20 is totally unusable for me, I was very angry and downgraded to 0.18... But when YouTube will change their interface, what options remains for me? Maybe I can find any other player? Or write another, "very tiny and simple, without bloatware" app? Like newpipe was sometime... |
Will be fixed in 0.20.1 |
the latest release was a total gnome move. Can someone please fork this project before it gets out of hand? |
You're a Github user. Be proactive and use the Fork button. :P |
@opusforlife2 is this a nice way of saying, go fork yourself? :D |
That is genuinely hilarious. I bow to your comedic genius. xD |
ClassicPress 2.0? |
While I don't disagree with the unified player, I can see the background queue being erased as being a problem. I could see a user trying to enqueue a YouTube video or Soundcloud song in the background while listening to their queue and accidentally erasing the list. The background player, i.e. the one with no video, would benefit from not being unified with the foreground players (fullscreen and popup) in this scenario. |
I like the "add to playlist" button |
Originally this issue was called "Unified player should be optional", not "Option to have separate queues". There is no need to have multiple queues for the usage scenario described in this issue. In v0.19 the background audio queue was totally unaffected by the video playback, and videos were never added to any queue when you simply watched them in NewPipe. If this behavior was available in 0.20 (as an option) instead of being completely removed, there would've been no need to come up with convoluted workarounds like the one described in #4460 (comment) (that doesn't really solve the actual problem described here). |
Yep, you are right |
This issue was never about "watching them later", it's about watching them immediately, without any changes to existing background queue. Please read the "Steps to reproduce the bug" section again. |
I don't. Did you even read my comments? I don't need video to be added to any queue when I simply press the Play button and watch it. I don't need any media notifications while it's playing either. This issue is not a feature request, because the feature already existed in NewPipe, and we know exactly how it's supposed to work. |
I'm sorry, @opusforlife2 , but I have to change the title of this issue back to the original one. It seems like new title creates more confusion and leads to people suggesting workarounds that are unrelated to the actual issue. |
It's not true.
You don't know how all the code works. It's not about what you want it's about how to make the player working. Notification is the only solution for playing something in android in a service.
It's not true too. Did you ever used NewPipe 0.19 before? When popup and background players work both notifications appear. Two notifications, not one. In case of three players there should be three notifications at a time. Did you understand or not? If not, ask your questions. Maybe by understanding the underlying changes you'll be able to find a solution for your problem |
I would say that it is possible to have separate queues but one service/notification. But someone needs to find a way to control all three queues in a comfortable way. That's hard. Let's say you're playing something in main player. Then you want to play something in background player. Main player pauses but UI is visible (play button and others, video is shown instead of thumbnail). Notification changes it's content to background queue instead of main player. Then you choose to play something in popup. Same as with previous example. But I don't know how to access background queue. probably via additional button in mini player at the bottom. So one button in mini player switches to background queue after pressing. Popup and main player still shows a picture of the paused video and it's controls but sound plays from backgroud queue. And click on notification will guide you to background queue. Whenever you play something within other player like popup or main, queue switches to it as well as notification. For this to work you need to have two cached player's UI for popup and main so they can coexist. Actually all this idea is not hard to implement. Of course this behavior should be optional. |
Yes, I don't know how the code works, but I do know how NewPipe behaves now and how it differs from pre-v0.20 behavior. My reference is this - https://github.com/XiangRongLin/NewPipe-preuinified , since it's still maintained and works properly on my device. This version does not display any notifications when a single video is being played in foreground, and, more importantly, this does not affect the background player queue/notification in any way. Also, I never said anything about the popup player, since I have no experience using it.
As I already said, at its core this issue was never about having control over multiple queues, since they are not needed for the usage scenario described in this issue. |
thanks @nbmrjuhneibkr for sharing Newpipe-preunified's repo. Is that repo updated to contain the latest fixes for playing youtube i.e. does youtube videos play in your device on that app?? |
@yashpalgoyal1304 It's NewPipe 0.19.8 + NewPipeExtractor 0.20.11 (and maybe some additional fixes). It is fully functional (at least for now). |
What do you guys think about this solution? #5864 (comment) |
Oh, hallelujah! At last someone made that fork! |
this is also the side-effect of unified player rework, right?: #5304 |
So, where is a resolution for this issue now at (2 years later)? I've read multiple related issues and still have no idea why the separate queues for foreground\background players had to be broken unconditionally. This really limited valid use cases of the app a lot. |
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
This comment was marked as spam.
Mod note: don't quote unnecessarily. I did not know this fork was a thing. I have been using the old NewPipe Legacy app alongside standard NewPipe just to avoid the very problem of not interrupting my existing queue when someone sends me a video. |
Checklist
Steps to reproduce the bug
New "unified player" makes the app useless if I want to use background player independently from the video player. I'm filing this as a bug, because this is something that worked perfectly fine in v0.19, but is completely broken now due to some questionable features being implemented in v0.20.
Actual behaviour
When I have tracks in the background queue (either playing or paused), and then try to watch a video in NewPipe, my background queue (which is valuable user data) gets erased without any warnings. This is default behavior in 0.20. The option "Ask for confirmation before clearing a queue" (which is disabled by default) adds a warning (that sometimes doesn't work - #4459 ), but does nothing to solve the actual problem. The fact that this option even needs to exist now is a clear indication that current implementation is wrong.
I have music and podcasts in my background queue pretty much 24/7, which means that I can't use NewPipe as a video player anymore without constantly losing my queue. And combined with #4458 this means that now I can't even open a video from NewPipe in another player without losing my background queue.
Expected behavior
There should be an option to use background player independently from the video player, just like in previous versions. These players serve a different purpose, and may be used for very different types of content.
Device info
The text was updated successfully, but these errors were encountered: