Collaborators tweaks after review

[pmaier1]

Moved to https://github.com/owncloud/enterprise/blob/master/product/oc10/index.md#collaborators

Tasks

[butonic]

Linking this here because the university of augsburg implmentet sharing on top of NFSv4 ACLs back in 2016. On slide 33 they present their mapping of oc CRUDS permissions to ACLs.

[pmaier1]

Feedback from first testing

• Opening left sidebar moves the whole screen (including e.g., the account indicator on the top right) => feels strange => intended? => Sliding animation of left sidebar too slow #1904

• Logout/Exit does not log out => works for me /Vincent

• The currently logged in user's name should be shown as "You" (e.g. as Owner) => https://github.com/owncloud/phoenix/issues/2062

• File list

  • Clicking on a file does not select it but highlight => intended? => yes, solved with @LukasHirt
  • Views are missing
    • Shared with you
    • Shared with others
    • Shared by link => Shared with link files section #2058 deferred

• Considerations UX fheidecke Sharing UI not guiding thru the process #1640 => very valid, IMO

• Pending Shares does not work and can break Phoenix => "Pending Shares" breaks Phoenix #1667

• Adding/modifying collaborators

  • Too less visible separation, focus and guidance between the elements => Confusing layout in collaborators UI #1908 Role interaction not consistent between collaborators and link share #1907
  • Roles: Need better explanation text: Collaborators panel UX fixes #2059
  • Roles: Explanation text, when already selected, should appear in the same edit box of the picker's
  • No way to undo changes instead of saving them
  • Bug: Switching from "custom role" to another role does only reload the permission checkboxes if you save
  • Custom role: Making changes after saving does not enable the save button again; can also happen when changing from "custom role" to another role
  • Single file: "Custom role" shows "can change" instead of "Edit" Custom role entry useless for file shares #1961
  • You can't find users by their user name if a display name is set
  • You can't find users by their mail address
  • "Extra field to display in autocomplete results" is not respected
  • Selected collaborators for adding should also have avatars and better styling for the delete icon
  • Federated sharing does not work => Federated sharing #2050 deferred
  • Guest sharing does not work => Guest users support #2057 deferred
  • Role switcher should have a default => actually the default single perms are configurable in the backend but as we don't have a mapping to roles there yet I would take "Viewer" as default for now
  • If you edit a collaborator and set the same content than before, it shows "Save" (e.g. have a collaborator as "Viewer", click on the dropdown and choose "Viewer" again)
  • If you edit a collaborator, make a change, then switch to another tab (e.g. "Links") and switch back to "Collaborators" => locks the UI

• Autocomplete

  • Existing collaborators are still shown => can even (try to) add existing collaborators
  • Avatars are not shown
  • Additional information (e.g. "Extra field to display in autocomplete results") is not shown

• Collaborator list

  • Reshare indicator appears even if it's not a reshare
  • "Custom role" is shown for "Editor/Viewer + can share"
  • Do we need a count for Collaborators AND users AND groups??
  • Trash bin icon to remove a collaborator maybe not appropriate?: Collaborators panel UX fixes #2059
  • Closing an "edit" dialog without saving may cause human mistakes => obsolete
  • (+) symbol to expand collaborators is not intuitive: Confusing layout in collaborators UI #1908 => obsolete
  • If reshare is not possible (e.g., disabled or no permission) users can still add collaborators but it fails
  • When the list takes some time to load, there's no spinner/indicator
  • Avatars are dummies
  • A reshare recipient can't recognize the resharer

• File list

  • The user can't conduct any actions for the current folder (e.g. add collaborators): File actions for the current folder #2061 deferred

• Collaborators can't recognize their role/permissions => Show role + explanation text somewhere

[PVince81]

Found in ee3b72e today:

• in "Shared with me" and "Shared with others", assuming we want the same design like in OC 10:
  • clicking on a folder must redirect to the actual folder in the "All files" view
  • a trash icon must be available to "unshare from self"
  • actions for accept/deny for pending shares (to verify)

=> raised as #2077

[LukasHirt]

actions for accept/deny for pending shares (to verify)

Should be implemented. Needs double-check.

[PVince81]

I've raised tickets for all the items. @pmaier1 you might want to review them and provide more information there in case it's not clear enough.

[PVince81]

• Add a margin bottom to "You don't have permission..."

[PVince81]

• Collaborator autocomplete must clear text field after selecting

[kulmann]

• In the "new collaborator" panel, selecting a user share excludes a group share with the same name from the list of candidates. Need to check for shareWith AND shareType, not only shareWith
• fix hr tags / separators in collaborator forms (maybe remove some, fix overflow)