Position of edit options in sharing menu confuses user #18163

Closed
michaelstingl opened this Issue Aug 10, 2015 · 24 comments

Projects

None yet
@michaelstingl

Based on user feedback, I would like to suggest to better structure the sharing menu. It is not clear at first sight, where the create, change, delete buttons belong.

Problem:
files_-_owncloud

Re-structured sharing menu:
files_-_owncloud

@jancborchardt What do you think?

@DeepDiver1975
Member

with 8,2 sharing will be move into the sidebar. UX will change.

@jancborchardt @PVince81 @blizzz screenshot of WIP at hand?

@karlitschek
Member

Independent if the sharing is in the dropdown or in the sidebare, a better grouping of the checkboxes and options is probably a good idea :-)

@DeepDiver1975
Member

Independent if the sharing is in the dropdown or in the sidebare, a better grouping of the checkboxes and options is probably a good idea :-)

thats the background of the request to see a screenshot of the side bar 😉

@jancborchardt
Member

How it should look like is the second screenshot of #3224

@MorrisJobke @PVince81 @owncloud/designers we can use the same bubble popover we use in #17709 (comment)

@jancborchardt jancborchardt added this to the 8.2-current milestone Aug 11, 2015
@MorrisJobke
Member

Example of wrong behaviour:

image

@PVince81
Collaborator

@MorrisJobke probably missing a white-space: nowrap or &nbsp.

I'd vote for a quickfix here if really needed and add the dropdown when in the sidebar. Because a dropdown in a dropdown isn't too nice (and detecting auto-close mouse click outside of dropdowns is such a pain)

@MTRichards
Contributor

Quick fix is ok for now, we just need it into 8.2. then sidebar for making it better as we have time, which might be 8.2 though probably not. Thank you!

@MorrisJobke
Member

The behaviour is already fixed in 8.1.0 - cc @CaptionF

@blizzz
Contributor
blizzz commented Aug 12, 2015

@MorrisJobke sure? current stable8.1 gives me:

screenshot

@MorrisJobke
Member

@blizzz Yes. Looks good. Checkbox and label are no longer on different lines. That was the problem that @CaptionF described.

The additional problems described in this ticket aren't fixed, but the screenshot I posted is fixed ;)

@jancborchardt jancborchardt added bug and removed enhancement labels Sep 15, 2015
@cmonteroluque
Contributor

move to 9.0 then for the remaining issues

@cmonteroluque cmonteroluque modified the milestone: 9.0-next, 8.2-current Sep 22, 2015
@jancborchardt
Member

@PVince81 @MorrisJobke @owncloud/designers for 9.0 we should actually move all the options (can edit, share etc) in a dropdown which is next to the delete function. The dropdown will have an indicator icon if that person can only read (eye, icon-toggle), or write as well (icon-rename).

@jancborchardt
Member

With dropdown, I mean in the same style as the file actions.

@PVince81
Collaborator

@jancborchardt sounds good.

@jancborchardt jancborchardt was assigned by bboule Jan 4, 2016
@bboule bboule removed the green-ticket label Jan 4, 2016
@bboule
bboule commented Jan 4, 2016

Hey guys so it seems like this is cosmetic however it is listed as a green ticket @michaelstingl this has an OC9 milestone I assume this is ok?

@Xenopathic
Member

The share options are now in the sidebar, although the options still don't have a very good layout:

screenshot_2016-01-04_19-35-07

@jancborchardt
Member

@PVince81 or @MorrisJobke let’s work on using a 3-dot dropdown menu here at the Nuremberg meetup. We should use the same as for the actions menu in files, but I had some problems when playing around with it.

@jancborchardt jancborchardt modified the milestone: 9.1-next, 9.0-current Feb 23, 2016
@michaelstingl

With 8.2.3, the can-edit-dropdown-arrow still gets separated from "can edit":

files_-_owncloud_enterprise_edition_and_skype_and_new_case_ _mstingl_owncloud_com__kerio___2772_messages_

@PVince81
Collaborator

Ok, let me throw in a quick   for 9.0.1 there.

@PVince81
Collaborator

Boo, even the checkbox can separate itself sometimes...

@PVince81
Collaborator

Turns out it was bit more work, but ok, PR is here #23386.
This will keep the checkboxes together.

@michaelstingl

I've got feedback from another user and I want to forward his input to this discussion:

It would be much more seamless to have this tabular, i.e. like this:

document1

@michaelstingl michaelstingl reopened this Apr 8, 2016
@jancborchardt jancborchardt was unassigned by PVince81 Jun 30, 2016
@PVince81
Collaborator

The tabular view is even less readable and would require sticky headers when you scroll down so you don't lose sight. I think this is a no go.

Since the original issue is fixed I'm closing this.

@michaelstingl feel free to reopen if you think otherwise

@PVince81 PVince81 closed this Sep 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment