-
Notifications
You must be signed in to change notification settings - Fork 53
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
Distinguish between local and non-local files with thumbnails #643
Comments
It works here with samba, showing or not thumbnails on the other PC following setting in PCMan, maybe I miss something. |
@stefonarch, the code does absolutely nothing. |
Here, the problem is distinguishing between local and remote files. A native file may be remote (like a file in Google drive, mounted locally by I tested by adding GLib's There's another GLib's attribute |
I think the safest way of giving sense to that option is tolerantly considering "local" as "native". There is a good reason: most external thumbnailers — if not all — only work with native paths and, for example, can't make thumbnails inside EDIT: Tested it but didn't like it. The above-mentioned reason seems good but, for example, having thumbnails with google drive isn't good. |
The code relies on the attribute `G_FILE_ATTRIBUTE_FILESYSTEM_REMOTE`; whether it's set correctly or not is the responsibility of GLib/GVFS. Closes #643 NOTE1: To be on the safe side, recompile all apps that are based on libfm-qt — but I didn't :P NOTE2: I haven't seen an external thumbnailer that works with remote files. So, this is mostly about libfm-qt's internal thumbnailer.
After 3 years, I didn't find a way better than using what GLib proposed, namely, |
I tested with ftp, looks working. Indeed toggling the option doesn't change anything. |
It isn't supposed to reload anything (if you mean that). Either the view should be reloaded or the directory should be changed after toggling the option. |
I meant remote thumbnails were always generated, with or without setting a hook at that option. |
Oh, you mean before the patch. Yes, remote internal thumbnails were always made. |
My approach failed (→ #1004 (comment)). Since most parts of libfm-qt's code are optimized by using asynchronous info queries, a synchronous querying of filesystem info is a bad idea, while enabling of the setting about local-only thumbnails doesn't seem achievable asynchronously. |
The patch does it only by a single info query for the parent directory. So, it shouldn't affect the speed. Also, the setting about local-only thumbnails is enabled in this way. Closes #643 and fixes #1006 NOTE: To be on the safe side, recompile libfm-qt based apps — although the LXQt apps don't seem to need that. REMINDER: Add an option to lximage-qt for local-only thumbnails if this PR is merged.
The patch does it only by a single info query for the parent directory. So, it shouldn't affect the speed. Also, the setting about local-only thumbnails is enabled in this way. Closes #643 and fixes #1006 NOTE: To be on the safe side, recompile libfm-qt based apps — although the LXQt apps don't seem to need that. REMINDER: Add an option to lximage-qt for local-only thumbnails if this PR is merged.
libfm-qt has an option to make thumbnails only for local files but that option has no effect for now (pcmanfm-qt inherits it in its Preferences dialog).
The text was updated successfully, but these errors were encountered: