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

Internal / private link #12877

Closed
4 tasks done
tobiasKaminsky opened this issue Dec 6, 2018 · 44 comments
Closed
4 tasks done

Internal / private link #12877

tobiasKaminsky opened this issue Dec 6, 2018 · 44 comments

Comments

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Dec 6, 2018

There is a feature request to have the internal/private link (clipboard on the right) also on android.

image

I discussed this with Jan and he is not very satisfied about the current situation.
Also Marie mentioned that she by accident copied this link to another person and this one could not open the file (as not shared and not even on server).

So, how could this get improved? Once we have a decision, this can also be done the same on all clients.

@nextcloud/designers
@marinofaggiana
@camilasan: newest 2.5.0 has this in option menu: "copy private link" / "copy public link"

@MorrisJobke
Copy link
Member

cc @rullzer @nickvergessen

@jancborchardt
Copy link
Member

jancborchardt commented Dec 21, 2018

We need to reduce the confusion and get rid of this link distinction:

  • We should get rid of the extra "internal link", especially because the icon is being confused for being an actual link
  • The link in "Sharing" could just be enabled when you click "Copy" for the first time or open the 3-dot menu, no distinction needed for the first one you create.
  • When you share any link to a file or folder with someone who has an account on that instance, they will see their general logged-in view instead of the share link view.

What do you think?

@jancborchardt jancborchardt added this to the Nextcloud 17 milestone Dec 21, 2018
@jancborchardt
Copy link
Member

Also ref issue on Android: nextcloud/android#795

@juliusknorr
Copy link
Member

We should get rid of the extra "internal link", especially because the icon is being confused for being an actual link
The link in "Sharing" could just be enabled when you click "Copy" for the first time or open the 3-dot menu, no distinction needed for the first one you create.

There is still a use-case for the internal link. Users want to share a file with a user (not via a share link) and then use the internal link as a reference in emails/chat/any other communication.

When you share any link to a file or folder with someone who has an account on that instance, they will see their general logged-in view instead of the share link view.

👍 That would be a nice improvement. You should not end up on a public page for a file if you are logged in and can access it though the regular files app.

@jancborchardt
Copy link
Member

There is still a use-case for the internal link. Users want to share a file with a user (not via a share link) and then use the internal link as a reference in emails/chat/any other communication.

Yes, I get the use-case of the internal link. But doing it via a share link which automatically detects if the user viewing it has an account fulfills the case much more seamlessly, while not messing with the overall UX.

@tobiasKaminsky
Copy link
Member Author

The only downside with this is that you have to create a new share link, explicitly for this user. You cannot "randomly" use the same link for many users:

  • I write a mail to whole company and want to show my cool xmas tree
  • I now can put it in a company-wide shared folder and link to the file

How would I do this then?

(overall I really like the idea that server is smart and handles sharing for you, like it is now with sharing, where you get one contact and not 3-4x the same contact, but with different sharing methods)

@jancborchardt
Copy link
Member

@tobiasKaminsky I'm not sure I understand it –why wouldn't you be able to use the same link for many people, whether they are on the Nextcloud or not?

@juliusknorr
Copy link
Member

You might want to share a private link without creating a public share link.

@jancborchardt
Copy link
Member

jancborchardt commented Dec 23, 2018

You might want to share a private link without creating a public share link.

Yes, but my point is that a "Share link" is not "Public" as it’s not guessable.

If that’s not the case, then I have another proposal (mockup):

internal link

Yes I know it adds yet another entry there, but it’s way better than having the "copy" icon so present on the top as it is right now. Also we have Share menu cleanup #12178 to rework the menu entries.

And the idea is that if you send the link to someone who has an account (or maybe even who via a different link has different permissions), they will be presented with the "highest" view possible to them. So if you paste an internal link on a forum, no one will see anything. But at the same time, if one of the collaborators on a folder opens a "Read-only" link, they will see their logged in view where they have all permissions.

@tobiasKaminsky
Copy link
Member Author

why wouldn't you be able to use the same link for many people, whether they are on the Nextcloud or not?

Hm. This might work, so I can just create a "public share link" and if someone on the same cloud is following the link, then he would not see the "public share page", but instead the regular cloud view with the desired file.

Then I guess, people might get confused. Especially if you create this link as readonly, but users on the same cloud can edit it (as their share is editable).

(maybe I am thinking too complicated)

(( merry xmas ))

@jospoortvliet
Copy link
Member

People can essentially already just copy the url in the browser to get the 'private' link, right?

Now we can have multiple links (and hopefully soon name them) it might make sense to have a 'internal' link on top of the list... It is annoying that it takes a spot but it is less confusing for users.

Of course, at the same time, we should let public links work smarter - as in, automatically become interal share links for logged in users if a file is already shared or allow users to make it a share in one click.

@jancborchardt
Copy link
Member

Alright, so who is up for working on the implementation? @juliushaertl @skjnldsv @violoncelloCH @kevgathuku maybe? ;) No rush, we could also set it to 17.

@jancborchardt jancborchardt added design Design, UI, UX, etc. feature: sharing labels Dec 27, 2018
@jancborchardt jancborchardt self-assigned this Dec 27, 2018
@MorrisJobke

This comment has been minimized.

@tobiasKaminsky

This comment has been minimized.

@MorrisJobke

This comment has been minimized.

@tobiasKaminsky

This comment has been minimized.

@jancborchardt

This comment has been minimized.

@tobiasKaminsky
Copy link
Member Author

So what is the conclusion here? @jancborchardt can you make a mockup, so that we all know what we want to have?

@jancborchardt
Copy link
Member

The proposal and mockup at #12877 (comment) still stands. :)

@tobiasKaminsky
Copy link
Member Author

Conclusion: move it into share list next to "add to collections", etc. and name it "Share internal".

@jancborchardt
Copy link
Member

What @tobiasKaminsky said but with the wording "Copy internal link". :)

@tobiasKaminsky
Copy link
Member Author

…and on mobile you also need to have a possibility to share ;-)

@tobiasKaminsky
Copy link
Member Author

same like "link creation" on android:

  • copy to clipboard
  • possibility to send link to any other app

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2019

@tobiasKaminsky unity on the whole column.
All the other entries have this button on the right. Clicking the whole entry is also not discoverable as it doesn't have any feedback nor does look like a button.

I'll suggest you do the same as it's weird to have some entries with the action on the right and then one where the full row is clickable :)

@tobiasKaminsky
Copy link
Member Author

Thanks for explanation.
I guess that this can be a bit different between our platforms.

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2019

I guess that this can be a bit different between our platforms.

I would say no :p
Let's be consistent.
Capture d’écran_2019-11-01_12-16-39

@skjnldsv
Copy link
Member

skjnldsv commented Nov 1, 2019

I'll suggest you do the same as it's weird to have some entries with the action on the right and then one where the full row is clickable :)

Actually, let me backpedal here ^^
I got confused because your screenshot from May look like our old sidebar, so it really have this "website" vibe. I'm guessing if you go with proper material design, the full entry is expected to be clickable.

Is there an opened issue for how the other entries differs from what we currently have? (like new link share... )
Also, your internal link button does not copy to clipboard but open the sharing menu of android, we should be clear about what it does.

Opening the sharing is good I'd say, but then I would not have the row called "copy internal link", but "Share internal link" cc @jancborchardt

@tobiasKaminsky
Copy link
Member Author

Is there an opened issue for how the other entries differs from what we currently have? (like new link share... )

Yes, #16228 and #15752

@tobiasKaminsky
Copy link
Member Author

"Share internal link" cc @jancborchardt

This is a good idea!

@tobiasKaminsky
Copy link
Member Author

@marinofaggiana on iOS this is missing completely, please add it on bottom just like on server.

@tobiasKaminsky tobiasKaminsky added high and removed 1. to develop Accepted and waiting to be taken care of 2. developing Work in progress labels Jan 16, 2020
@rullzer rullzer modified the milestones: Nextcloud 17.0.4, Nextcloud 17.0.5 Mar 11, 2020
@rullzer rullzer modified the milestones: Nextcloud 17.0.5, Nextcloud 17.0.6 Mar 23, 2020
@skjnldsv skjnldsv removed this from the Nextcloud 17.0.7 milestone Apr 27, 2020
@marinofaggiana
Copy link
Member

done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants