Skip to content

Prompt for default screenshot folder when it is unset#8581

Merged
FreeTubeBot merged 1 commit intoFreeTubeApp:developmentfrom
absidue:screenshot-default-prompt
Feb 3, 2026
Merged

Prompt for default screenshot folder when it is unset#8581
FreeTubeBot merged 1 commit intoFreeTubeApp:developmentfrom
absidue:screenshot-default-prompt

Conversation

@absidue
Copy link
Member

@absidue absidue commented Jan 24, 2026

Pull Request Type

  • Bugfix

Related issue

Description

Currently if the user disables the ask for save folder setting in the screen shot settings but doesn't chose a default folder, we fall back to the pictures folder. That doesn't work in the flatpak build though, as we don't grant it access to the users picture's folder (it's not needed for most FreeTube functionality and the screenshot setting is disabled by default).

To fix that FreeTube will now prompt the user to choose a folder when they take a screenshot if they haven't previously chosen one or it doesn't have write access to the folder that is configured, the latter can occur if the user copies their settings between machines or intentionally revokes access to the previously chosen folder through flatseal. If the user clicks cancel in the folder chooser dialog, FreeTube doesn't save the screenshot and the user will be prompted again the next time they take a screenshot.

Testing

  1. Open your settings.db file in a text editor and delete the screenshotFolderPath line (if it exists)
  2. Open FreeTube
  3. Enable taking screenshots
  4. Disable prompting for the folder
  5. Open a video
  6. Take a screenshot
  7. You should be prompted for a save folder
  8. If you selected a folder the next screenshot you take should get saved to that location without another prompt.

Desktop

  • OS: Windows
  • OS Version: 11

@FreeTubeBot FreeTubeBot enabled auto-merge (squash) January 24, 2026 12:38
@github-actions github-actions bot added the PR: waiting for review For PRs that are complete, tested, and ready for review label Jan 24, 2026
@FreeTubeBot FreeTubeBot merged commit 92c2908 into FreeTubeApp:development Feb 3, 2026
6 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 3, 2026
@absidue absidue deleted the screenshot-default-prompt branch February 3, 2026 06:42
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