+1 for me :) But I am not the maintainer :)

+1 for the idea! Needs more screenshots. :)

lgtm overall. but I wonder about the naming choice. "Shared" is confusing, to me personally at least. Why not call it "Remote" or "Network"? The reasoning is that "shared" for me is only LAN/WLAN shared stuff, while "remote://" allows arbitrary remote links. I use it to connect via (S)FTP to machines on the internet e.g.

In D8434#162727, @mwolff wrote:

lgtm overall. but I wonder about the naming choice. "Shared" is confusing, to me personally at least. Why not call it "Remote" or "Network"? The reasoning is that "shared" for me is only LAN/WLAN shared stuff, while "remote://" allows arbitrary remote links. I use it to connect via (S)FTP to machines on the internet e.g.

+1

In D8434#162727, @mwolff wrote:

lgtm overall. but I wonder about the naming choice. "Shared" is confusing, to me personally at least. Why not call it "Remote" or "Network"? The reasoning is that "shared" for me is only LAN/WLAN shared stuff, while "remote://" allows arbitrary remote links. I use it to connect via (S)FTP to machines on the internet e.g.

I first thought of using "Network" but this is already used as label for "remote://" url. "Remote" can be a option since it is not used yet. But "Shared" came from MacOS images posted in another review, looks like is is used by "Finder".

Well, but if we use Network for remote:// already, then the group should also have this label, no? I don't see an issue with this, really. On the contrary - maybe we could in the future remove the remote:// link and let the category header react to a click, such that it will show you e.g. all remote devices. That would remove the duplication then.

But thinking 5s more about it, I personally would say that using "Remote" for the category would be OK for now.

In D8434#162786, @mwolff wrote:

Well, but if we use Network for remote:// already, then the group should also have this label, no? I don't see an issue with this, really. On the contrary - maybe we could in the future remove the remote:// link and let the category header react to a click, such that it will show you e.g. all remote devices. That would remove the duplication then.

I think duplicate

In D8434#162002, @ngraham wrote:

+1 for the idea! Needs more screenshots. :)

screenshot added

+1 on Remote.

We really need to remove the duplicated Places test that's the header for the whole widget, though.

Can you add "Depends on DXXXX" for each other patch that this requires?

@dfaure?

@renatoo Next! :-) Time to update this one, too.