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

nemo-file.c: "Always" show preview now literally means "always" #3467

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jmaebe
Copy link

@jmaebe jmaebe commented Oct 3, 2024

gvfs reports that we should never show thumbnails for files on mtp devices such as phones. Allow users to override this when they select "always". If it gets too slow, the file size filter can still be used to further limit for which files previews are generated.

Add a new "Recommended Only" setting that uses the underlying gvfs recommendation.

Based on https://gitlab.xfce.org/xfce/thunar/-/merge_requests/348

resolves #3291

gvfs reports that we should never show thumbnails for files on mtp
devices such as phones. Allow users to override this when they select
"always". If it gets too slow, the file size filter can still be used to
further limit for which files previews are generated.

Add a new "Recommended Only" setting that uses the underlying gvfs
recommendation.

Based on https://gitlab.xfce.org/xfce/thunar/-/merge_requests/348

resolves linuxmint#3291
@jmaebe
Copy link
Author

jmaebe commented Oct 3, 2024

One known issue: the added preview option for showing previews only if recommended by gvfs shifts existing saved preview options (due to the entry added in the middle of the enum). I'm not familiar enough with gtk and how it saves/loads settings to know how to fix this. Adding the entry at the end feels out of place.

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.

No image/video preview with MTP connected mobile phone storage (Linux Mint 21.2 - victoria)
1 participant