Only start playing single item in history screen

codyit commented 1 week ago

What is it?

[x] Bugfix (user facing)
[ ] Feature (user facing)
[ ] Codebase improvement (dev facing)
[ ] Meta improvement to the project (dev facing)

Description of the changes in your PR

Holding an item in history and clicking "Start playing in the background" would now only add the current item in the play queue, instead of the full history which is usually massive.

Before/After Screenshots/Screen Record

N/A

Fixes the following issue(s)

Fixes # https://github.com/TeamNewPipe/NewPipe/issues/5586

APK testing

The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR. You can find more info and a video demonstration on this wiki page.

Due diligence

[ x] I read the contribution guidelines.

Stypox commented 1 week ago

There are people that rely on this behavior (me included) and last time we changed something we got a lot of angry users. During the refactor we will look into redesigning the long-press menu to allow for all of these different options to co-exist (starting without enqueueing everything and starting a playlist from a specific item), but we can't just change one thing without thinking about the whole picture. Sorry about it, and thank you for your contribution anyway.

Stypox commented 1 week ago

Oh wait, you are just doing the change for the history fragment. Then I think the change makes sense.