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

[Feature request] Make it possible to show volumes on the desktop #1192

Open
probonopd opened this issue Sep 3, 2020 · 11 comments
Open

[Feature request] Make it possible to show volumes on the desktop #1192

probonopd opened this issue Sep 3, 2020 · 11 comments

Comments

@probonopd
Copy link

probonopd commented Sep 3, 2020

Expected Behavior

There should be a way to show volumes/partitions on the --desktop view, like they already show up under "Devices" in the side panel of the the file manager.

Most desktops allow this. E.g., Xfce:

image

Current Behavior

Volumes/partitions cannot be shown in the --desktop view.

System Information
  • Distribution & Version: FreeBSD 12.1-RELEASE-p8
  • Kernel: FreeBSD 12.1-RELEASE-p8
  • pcmanfm-qt Version: 0.15.0
@probonopd
Copy link
Author

There could be an option to enable this alongside the existing options:

image

@probonopd
Copy link
Author

probonopd commented Sep 3, 2020

Unlike the existing options, what I am suggesting would not just create some .desktop file on-the-fly. Instead, it would add icons on the desktop as volumes/partitions come and go, without creating .desktop files for those. Those icons would have context menu items that could e.g., unmount/eject them, like it is already possible today under "Devices" in the side panel of the the file manager.

@probonopd
Copy link
Author

Looks like in desktopwindowmodel.c one would need to use modelFromPath(fm_path_new_for_uri("computer://")) somehow in addition to what is already there, so that the "normal" icons and the ones for computer:// would get shown. What would be the best way to achieve this?

@tsujan
Copy link
Member

tsujan commented Sep 5, 2020

Sorry for the late reply -- I was so busy.

First, this is a valid feature request.

Second, it seems that you were talking about a very old version of pcmanfm-qt or, perhaps, about the abandoned GTK version. We don't have desktopwindowmodel.c.

Third, the side-pane is completely different from Fm::FolderView. In the former, we deal with items; in the latter, with files and only files. That's the reason why Home, Trash, Computer,... are desktop entries.

Generalizing Fm::FolderModelItem to include non-files was my first thought before adding those items as desktop entries but it proved to be so complex and, especially, so dirty that I abandoned it.

If you can do it by writing a clean and concise code, your PR will be welcome.

@probonopd
Copy link
Author

probonopd commented Sep 5, 2020

you were talking about a very old version of pcmanfm-qt

Indeed, I am currently experiementing with my fork of 0.10.0 but I think the general gist of the ticket should still apply to the latest one.

What I was trying to say is that if I change the model that is used for the desktop, then the current code already does everything that is needed. The only thing is missing is some glue code that can combine 2 models into 1. Do you know a way to achieve this?

@tsujan
Copy link
Member

tsujan commented Sep 5, 2020

if I change the model that is used for the desktop,

If you take a glance at the latest code (or the code from last year or 2 years ago), you'll see that we don't use a different model for desktop. The original author simplified the code and it should remain that way.

Anyhow, in this case too (like in #1193), let's forget about words. A clean and relatively short code will be acceptable for this.

@coldReactive
Copy link

Also would like to see this possible with udiskie and udisks2 support maybe, to get away completely from gnome dependencies with lxqt.

@tsujan
Copy link
Member

tsujan commented Jul 4, 2024

... to get away completely from gnome dependencies with lxqt.

LXQt has no GNOME dependency. GNOME even doesn't have a desktop, in the true sense of the word "desktop".

@coldReactive
Copy link

... to get away completely from gnome dependencies with lxqt.

LXQt has no GNOME dependency. GNOME even doesn't have a desktop, in the true sense of the word "desktop".

gvfs will, however, get some gnome dependencies, such as gsettings-desktop-schemas

@probonopd
Copy link
Author

Things like libfm, gvfs are arguably deep in "Gnome land".

@tsujan
Copy link
Member

tsujan commented Jul 4, 2024

Using a fundamental library developed by the GNOME team doesn't mean having a GNOME dependency. GNOME is a DE based on GTK, while pcmanfm-qt and libfm-qt are Qt programs that use GLib for file management because GLib has proven to be superior to its Qt/KDE counterpart(s). GVFS extends GLib's functionalities a lot.

Anyway, that has nothing to do with this feature request.

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

No branches or pull requests

3 participants