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

Download action in share link header split in two #9933

Closed
jancborchardt opened this issue Jun 20, 2018 · 6 comments · Fixed by #12316
Closed

Download action in share link header split in two #9933

jancborchardt opened this issue Jun 20, 2018 · 6 comments · Fixed by #12316
Labels
Milestone

Comments

@jancborchardt
Copy link
Member

In the header of sharing, on the top right there’s the download actions:
screenshot from 2018-06-20 13-51-10

Now the design for this (and as it worked before) is:

  • Both text and 3-dot menu toggle are one element. The text is merely a label for the icon as the actions are all "related" to download (and Download shows in the menu).
  • The text "Download" is not displayed on mobile to save space for the title of the file, only the 3-dot-menu remains.

As it currently is:

  • It’s split into "header-primary-action" and "header-secondary-action" which is confusing.
  • The "Download" text does not vanish on mobile.

Now we could do the following to mix the approaches:

  • On mobile, only have the 3-dot icon and all actions in the 3-dot menu (primary action first) like in the intended design
  • On bigger sizes, having the primary action outside of the menu (like in the current way)
  • To make it less confusing, on desktop we need to make obvious that there’s a primary button, so we should probably add a white border (a la button style) around the text.

What do you think @juliushaertl @skjnldsv @nextcloud/designers?

@skjnldsv
Copy link
Member

Is this causing issues actually? Or is it just a nitpick to improve the design?

@juliusknorr
Copy link
Member

On mobile, only have the 3-dot icon and all actions in the 3-dot menu (primary action first) like in the intended design

👍

To make it less confusing, on desktop we need to make obvious that there’s a primary button, so we should probably add a white border (a la button style) around the text.

That sounds like a good approach. @nextcloud/designers

image

@skjnldsv
Copy link
Member

Ah yes, I did not understand the issue like that!
I'm all in as well :)

@skjnldsv skjnldsv added the 1. to develop Accepted and waiting to be taken care of label Jun 27, 2018
@jancborchardt
Copy link
Member Author

@juliushaertl yep, like that! :) But on bigger widths, the "Download" action then does not need to be in the 3-dot menu too when it’s already shown outside of it. Otherwise, want to open a pull request for it?

@MorrisJobke
Copy link
Member

@jancborchardt @skjnldsv Moved to 15

@skjnldsv
Copy link
Member

skjnldsv commented Aug 24, 2018

@MorrisJobke no, it's in #10838
At least the download being hidden on small width

@skjnldsv skjnldsv added 3. to review Waiting for reviews and removed 1. to develop Accepted and waiting to be taken care of labels Nov 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants