server icon indicating copy to clipboard operation
server copied to clipboard

Improvement to sharing design and flow

Open jancborchardt opened this issue 4 years ago • 9 comments

Currently we have several issues with sharing, across platforms:

  • It looks a bit cluttered, especially the 3-dot dropdown menu with all the actions is too much
  • It’s not clear to people when the share will be set & sent, leading to possible access control issues, and also 2 separate mails when you set a note
  • Names are often cut off because of the "can edit" with checkbox
  • It could be more consistent across platforms

Here’s a proposal to fix these issues which came out of a design discussion @FlexW and I had about improving the sharing flow for the Desktop client. We should of course make sure to keep this the same across all platforms. (It is to be seen as a mockup – of course the inputs and buttons should be the native elements of the platform.)

Share overview Share details Share details advanced
Share overview Link details vertical Link details vertical advanced

Mockup with slight interactivity at ~~https://design.penpot.app/#/view/b83aa940-a28e-11eb-bb49-81aaa3a367f9/b83aa941-a28e-11eb-bb49-81aaa3a367f9?token=9eUwuY6LzqqZ34pWFWSUnQ&index=0~~ https://design.penpot.app/#/view/b83aa940-a28e-11eb-bb49-81aaa3a367f9?page-id=b83aa941-a28e-11eb-bb49-81aaa3a367f9&index=0

Share overview

  • We add a subline to the entries where info on "View only" or "Can edit" or "File drop" is shown – this makes name and subline area actionable
    • For external shares of single files where there’s no editing, the "Can edit" function needs to be not shown (anything except txt, md, or office probably)
    • Even if there is only one option, we should still show the subline for feedback
  • The avatar needs to stay a separate actionable area for the contact options
  • Another direct action in the list is to "Copy link" because that needs to be quickly accessible.
  • On the right there’s a 3-dot menu icon which brings up a menu with quick actions for
    • Share details
    • Resend email
    • Delete share / Unshare
  • Going to "Share details" shows the detail page with further info
  • The little primary-colored bar on the top is a little experiment and would mainly be something for the desktop client to make the dialog look less dead.
  • Alternatively on mobile: Whole entry is tappable and opens the menu as a bottom sheet, in this case with
    • (Direct switch of View only / Edit / File drop if possible?)
    • Share details
    • Resend email
    • Delete share / Unshare

Share detail

  • Ideally, there is a quick slide left/right animation whenever one navigates in and out of this detail view so people have a sense of place.
  • The header part needs to indicate both which file it is about, and if it is a share link or which person. In this mockup it is done by changing the header – this works for Desktop, for the other platforms we need to adjust.
  • This opens with the relevant settings – when there is an expiration date set or enforced, the "Advanced settings" is already open
  • Here all the share settings are listed, in this order:
    • View only / Edit / File drop
    • Note to recipient
    • Set password
    • Set expiration date
    • Share label
    • Delete link / Unshare
  • Clicking the "Share & copy link" button automatically copies the link as expected. For shares to people, this could say "Share & send mail"

@nextcloud/designers and the leads @tobiasKaminsky @rullzer @marinofaggiana @mgallien @FlexW @LukasReschke @karlitschek

jancborchardt avatar Apr 22 '21 11:04 jancborchardt

Related https://github.com/nextcloud/server/issues/12178 https://github.com/nextcloud/server/issues/8479

Shall we close the others @jancborchardt ?

skjnldsv avatar Apr 22 '21 12:04 skjnldsv

I think this issue is also related: https://github.com/nextcloud/server/issues/25123

@jancborchardt since your proposal and my proposal from the related ticket are quite similar, I would agree that your mockups totally make sense. :P I think It would be a huge improvement from a user experience point of view if we could keep it like this over all platforms.

bes1002t avatar Apr 22 '21 21:04 bes1002t

Just adding my two cents from the related issue #22132 in case it makes sense to add a 'Direct Download' option to the 'Advanced' screen. After months of use we're pretty happy just appending the '/download' as and when needed so I'm less fussed now than I was when I first raised it. Still, it seemed worth mentioning in case anyone else can see the benefit.

bbceg avatar Jun 11 '21 16:06 bbceg

Hello there,

from the many user feedbacks I collected of the previous months, I greatly support these ux ideas. I think that would make it well easier for people to understand who they share with things and also who can do what. It's the core of nextcloud and anything that can improve this experience improves directly the value of the overall experience.

Here are a couple of users feedbacks I observed frequently and which could be of interest :

  • "internal link" : I observed 50% of the time (if not more) people are getting confused with it. At the moment it's gray and at the bottom which gives it much less affordance than the "share link". So what I've seen is : users click on "share link" (because for example they want to share it in a conversation) and share that... with people from their organization who have access to it (!). This effect in various frustrating experiences.

Example of a real situation I've seen many times : someone share an external link (instead of internal) to a document to their team before an important group call to work on a document. The call starts, everyone connects to the document to work, and of course they expect to be able to work on it because they're part of the same team and they know they can edit it. But what happens is the "shared link" didn't have "edit mode" allowed... Then you get in that situation when no one can edit the document at that important moment and most people don't understand what happens... Then maybe someone manages to open the document in a regular way and says 'hey it works for me, why not for you?'... You get the scene... Let that happen 2 times and we get a call "your software is not reliable, we lost two meetings that way, we're thinking about going back to GDrive". 😔

A suggestion I can share is that maybe there shouldn't be a difference from the beginning and you should choose after. First you click on "share a link" and only then the UX let you understand whether you want to get an internal link or you need to create an "external" link. A bit of work on copywriting could also help, I'm not sure "internal" and "external" is that easy to get.

  • the field "share with users" : I've seen many people not understand this is actually a text field where you can enter something. They just don't see it and go directly to create a "link share".

If these two parts are of interest to you folks here, don't hesitate to tell me and I'd be glad to take some time to share some proper mockups.

cheers :)

ImaCrea avatar Sep 16 '21 14:09 ImaCrea

pull request: https://github.com/nextcloud/server/pull/28832

schiessle avatar Sep 16 '21 15:09 schiessle

pull request: #28832

Hello @schiessle thanks, I checked the pr but not sure I get everything there :/

ImaCrea avatar Sep 20 '21 17:09 ImaCrea

Probably the wrong place to bring this up, but, what about moderation to conversations? currently with a file I shared as an admin of the server, I'm unable to delete guest comments in the Chat interface of the sidebar.

sn0n avatar Mar 20 '23 01:03 sn0n

@sn0n thanks for mentioning this, could you please raise this as a separate issue via https://github.com/nextcloud/server/issues/new/choose - highly appreciated! 💙

AndyScherzinger avatar Mar 20 '23 08:03 AndyScherzinger

@jancborchardt I think that these flows are complicated enough that they deserve a big spacious dialog and even further breakdown of the sharing steps. Do you think we could further iterate on this design and try to do that?

marcoambrosini avatar Jun 16 '23 03:06 marcoambrosini

@jancborchardt can you do a briefing call / discussion with @Fenn-CS on this item. The mockups look good but are generic for all platforms so I expect some decisions will be needed on details/specifics, i.e. how this should be implemented on web in the right sidebar. I suggest we don't further iterate for the moment else we won't be able to implement it in time.

AndyScherzinger avatar Jul 06 '23 08:07 AndyScherzinger

Saving the mockup of advanced CRUD permissions from https://github.com/nextcloud/server/issues/30227#issuecomment-999601203

This could be put in the "Advanced settings" of sharing, as a "Custom permissions" checkbox which has subitems when checked. If it is checked and modified from the defaults, on the top there will be another entry "Custom permissions". This one does not show by default and is merely informative. One can then quickly switch back to more general "View", "Edit" or "File drop". Link details vertical advanced with permissions

jancborchardt avatar Aug 09 '23 14:08 jancborchardt