NewPipe icon indicating copy to clipboard operation
NewPipe copied to clipboard

Add a loop/replay button

Open theridon opened this issue 3 years ago • 7 comments

Checklist

  • [X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
  • [X] I'm aware that this is a request for NewPipe itself and that requests for adding a new service need to be made at NewPipeExtractor.
  • [X] I have taken the time to fill in all the required details. I understand that the feature request will be dismissed otherwise.
  • [X] This issue contains only one feature request.
  • [X] I have read and understood the contribution guidelines.

Feature description

This app has just about everything else, it's great. But for some reason there is no replay/loop button/functionality? Can we just get a simple button below the video that would let it replay videos, such as for listening to music in a car? Thank you.

Why do you want this feature?

So the app is better

Additional information

No response

theridon avatar Jul 17 '22 23:07 theridon

Rejected. See: https://github.com/TeamNewPipe/NewPipe/issues/6914

However, you can still use this functionality from notification player. Go to Settings>Appearence>Player Notification>4th/ 5th Action Button> Select Repeat

SameenAhnaf avatar Jul 17 '22 23:07 SameenAhnaf

I disable notifications from my apps, so I can't use that feature. Plus it's quite inconvenient to have to drag down the notification bar for such a basic, integral feature. Can we just get a button that's usable when normally using the app?

theridon avatar Jul 18 '22 01:07 theridon

I disable notifications from my apps

In version 0.20.0, Newpipe's player was rewritten and made wholly dependent on a unified player service, which needs a foreground notification to anchor it in memory.

I don't think we support use cases where notifications for the app are disabled. But I'll let a developer weigh in on this.

opusforlife2 avatar Jul 18 '22 04:07 opusforlife2

I wholeheartedly agree that it doesn't make sense for the looping functionality to be present only in the media notification, not the app itself. It's quite inconvenient. Like I stated in https://github.com/TeamNewPipe/NewPipe/issues/6914#issuecomment-899027896, adding a loop button in the player would be a good idea in my opinion.

triallax avatar Jul 18 '22 11:07 triallax

I don't think we support use cases where notifications for the app are disabled. But I'll let a developer weigh in on this.

Foreground apps work just fine if you disable their notifications ;-)

Stypox avatar Jul 22 '22 13:07 Stypox

I agree with Stypox's solution in https://github.com/TeamNewPipe/NewPipe/issues/6914#issuecomment-1190142193.

opusforlife2 avatar Jul 25 '22 18:07 opusforlife2

This is something that should be considered for the rewrite.

opusforlife2 avatar Aug 27 '23 16:08 opusforlife2