-
-
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
Make downloaded files nothing more than a cache so that all the features work #10542
Comments
The location for this should probably be Maybe there could be a dialog upon tapping Download to choose between 2 options:
|
This option in settings could be shown in a more simpler way. |
There could be two checkboxes as well: Videos and Audios. Some users may have only video or only audio player app installed on their phone. Or, one of their used apps don't offer the function available on Newpipe app. These checkboxes will be helpful for them. |
This would be wonderful. I currently use a podcast app to play downloaded videos. This is less than ideal for several reasons:
Please see this as mere brainstorming...I'm quite aware this all gets really tricky. |
That removes from the user the ability to choose where individual downloads go. I would understand having a checkbox like "Don't ask me again" for the Download alternatives dialog, but tightly coupling local playability with downloading to a specific location seems restrictive. Another alternative is two completely separate buttons. The existing Download button stays as it is, and an additional "Save/Cache/Mark for offline playback"/"Play offline" can be added. I would suggest replacing the "Add to playlist" button with this one instead. |
@foxjaw Instead of making multiple comments if you have something to add, you can edit your existing comment as many times as you like. |
This is good but I'd want to be able to download to open with VLC for example. Maybe as an option which is off by default. |
I'd advocate for two separate buttons:
This way, offline videos could also be put into the app's private storage's |
That's a tricky one. One straightforward answer to this is that we simply don't use the regular cache folder, but another one that persists across cache-clearing operations. This folder would only be deleted if the user specifically takes that action. However, that means offline videos would start filling up the App Data instead of Cache metric. If anything goes wrong (as it often does with filesystem operations), then the only way to get rid of all that potential junk is to clear the app data. Very non-ideal. I don't know if it's possible, but maybe the offline video folder could remain inside the regular cache folder, and using the cache-clearing operation from within the app would prevent that folder from being emptied. Or maybe show a dialog asking if the user wants to delete cached videos as well. This would also mean that clearing cache from the OS App Info menu would remove the entire cache including videos, but we could explain that in the app settings, and maybe in our FAQ as well. |
In that case, any filesystem operations going wrong would cause junk to be left around, bloating storage. |
@opusforlife2 Not if the videos are stored in a well-designated directory and there's no database about what videos are supposed to be there. The name of the file already designates what the video is and how to link it to the original one so that one is executed instead of the original one. One straightforward way would be:
The |
@foxjaw That information can be stored in the companion file. I wrote there |
Checklist
Feature description
Many features don't work on downloaded files, and there are several bugs already open for individual features (enqueue #9829, mark as watched #9334, already downloaded #7360), but the real problem is that downloaded files are considered a separate case with its own features that are independent from the rest of NewPipe.
Also, when playing a downloaded file, it stops playing if you switch to a different app, one can't play the audio only, etc.
If downloaded files were considered a cache (which obviously requires a lot of work), then all the feature would not have to be treated as a special case, with a different bug for each of them, and new features would take this case into account.
Why do you want this feature?
Before losing data connectivity, I want to download a bunch of videos, and then continue using NewPipe offline, with all the features still working (going to a channel, selecting a video to play, switching to another app, etc), as all the information NewPipe needs is available locally.
Additional information
No response
The text was updated successfully, but these errors were encountered: