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

Better primary visual on public pages #12316

Merged
merged 1 commit into from
Nov 7, 2018
Merged

Conversation

skjnldsv
Copy link
Member

@skjnldsv skjnldsv commented Nov 6, 2018

  • better visual for the primary button
  • hide the download button when the primary is shown

fix #9933
@nextcloud/designers

@juliushaertl should we always display the primary in the menu and only show it when mobile?
I mean, not only the download? 🤔
@nextcloud/designers any thoughts?

@skjnldsv skjnldsv added bug design Design, UI, UX, etc. 3. to review Waiting for reviews regression labels Nov 6, 2018
@skjnldsv skjnldsv added this to the Nextcloud 15 milestone Nov 6, 2018
@skjnldsv skjnldsv self-assigned this Nov 6, 2018
@MorrisJobke MorrisJobke mentioned this pull request Nov 6, 2018
29 tasks
@jancborchardt
Copy link
Member

jancborchardt commented Nov 7, 2018

(Screenshots please for design changes. ;)

sharing folder page

Looking great with the button style – much more call to action! 👍

(But there you can also see how strange the short folder names look left-aligned. ;)

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 7, 2018

should we always display the primary in the menu and only show it when mobile?
I mean, not only the download? 🤔

What do you think @jancborchardt :)

@jancborchardt
Copy link
Member

Sorry, I don't really get the question, could you rephrase? :)

It's ok like it is now. A possible change could be to:

  • Show the download icon left of the text in the button
  • Then on mobile not simply hide the whole button, but only the text so a circular primary download button is still directly shown.

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 7, 2018

I mean, always have the primary action outside AND inside the menu.
But only showing the primary when not in mobile view.

Right now we declare the download action twice and hide the menu via css :/

@jancborchardt
Copy link
Member

I mean, always have the primary action outside AND inside the menu.

That doesn’t make any sense though. An action should only be shown once.

But only showing the primary when not in mobile view.

I don’t get that? Not showing the 3-dot-menu at all?

I’m confused as to what is wrong with the current design:

  • On mobile, you see the 3-dot icon with menu: download, direct link, add to your Nextcloud
  • On desktop, the important action of "Download" is directly shown, with the others being still in the menu.

This is also what e.g. Material Design does.

@skjnldsv
Copy link
Member Author

skjnldsv commented Nov 7, 2018

Sorry, I wasn't clear enough 🙈
Currently: the first action is defined as primary and is hidden in mobile view. it will NOT appear in the menu in mobile view. The primary action and the menu are two different things.

I'll suggest we put the primary action back into the menu in mobile mode so we'll have:

  1. primary outside and not in menu in desktop view
  2. primary not outside BUT in the menu in mobile view.

@MorrisJobke
Copy link
Member

@skjnldsv Conflicts :/

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
@MorrisJobke MorrisJobke force-pushed the public-download-button-fix branch from eecf776 to f54b201 Compare November 7, 2018 15:31
@MorrisJobke
Copy link
Member

@skjnldsv Conflicts :/

Rebase was quite easy 😄

Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it 👍

@MorrisJobke
Copy link
Member

@juliushaertl should we always display the primary in the menu and only show it when mobile?
I mean, not only the download? 🤔

Let's keep it like that for now in the PR, merge it and discuss about this additional topic in a separate PR/issue.

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 7, 2018
@rullzer rullzer merged commit a8fb7e1 into master Nov 7, 2018
@rullzer rullzer deleted the public-download-button-fix branch November 7, 2018 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug design Design, UI, UX, etc. regression
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Download action in share link header split in two
4 participants