Skip to content

Add support for accessing network shares #24

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

hadess
Copy link

@hadess hadess commented Feb 7, 2022

This works, but the speed is pretty atrocious. I don't know what calls it's doing in the background, but it seems to have problems redrawing fast enough to keep the UI responsive.

@neatdecisions
Copy link
Owner

Thank you!
Indeed, to have an acceptable UX here we'd need a proper asynchronous processing when switching location.
It's in the pipe, I'll wait with merging this PR until it's done.

@hadess
Copy link
Author

hadess commented Apr 19, 2022

The first 2 commits could be merged straight away though.

Don't rely on backends providing more information than we requested. If
we don't ask for the file type, some gvfs backend won't provide it.
@hadess hadess force-pushed the wip/hadess/remote-support branch from 608f6d7 to ca47c21 Compare December 11, 2022 19:07
@hadess
Copy link
Author

hadess commented Dec 11, 2022

I've rebased the code, still has the same problems, the first 2 commits could still be merged ;)

One thing I've also noticed is that "Move to Trash" won't work on some remote filesystems, and that the code should check for G_FILE_ATTRIBUTE_ACCESS_CAN_TRASH being true before enabling that feature (and probably also G_FILE_ATTRIBUTE_ACCESS_CAN_DELETE if it doesn't already).

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.

2 participants