Skip to content
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

Change the default download behaviour to the better "Open in web browser" value #4683

Merged

Conversation

absidue
Copy link
Member

@absidue absidue commented Feb 16, 2024

Change the default download behaviour to the better "Open in web browser" value

Pull Request Type

  • Bugfix

Description

As discussed with the other maintainers, this pull request changes the default download behaviour to the objectively better "Open in web browser" option.

When the pull request that added the minimal "Download in app" option was introduced, it was done so with the expectation that it would be expanded upon not long afterwards. As it's now two years later and the in-app downloader hasn't been improved, we should definitely not default to it. While the "Open in web browser" option isn't great either, it is the lesser of two evils, as web browsers generally come with download managers, including being able to view progress, pause and resume downloads.

If users want to use the incomplete in-app downloader, they can still opt into it by changing the setting, we just will no longer use it as a default.

Desktop

  • OS: Windows
  • OS Version: 10
  • FreeTube version: 0.19.1

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) February 16, 2024 21:36
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 16, 2024
@FreeTubeBot FreeTubeBot merged commit 48e2f0c into FreeTubeApp:development Feb 17, 2024
5 checks passed
@github-actions github-actions bot removed the PR: waiting for review For PRs that are complete, tested, and ready for review label Feb 17, 2024
@absidue absidue deleted the fix-default-download-behaviour branch February 17, 2024 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants