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

Only show link share menu if needed #14280

Closed
wants to merge 1 commit into from

Conversation

juliushaertl
Copy link
Member

The menu for share links does not need to be opened by default any more since the copy icon has been moved out of it. Therefore it makes more sense to me to just open it if we enable password protection by default so the user can enter a password. Otherwise there is no need to open the menu.

@nextcloud/designers

@juliushaertl juliushaertl added 3. to review Waiting for reviews papercut Annoying recurring issue with possibly simple fix. labels Feb 19, 2019
@juliushaertl juliushaertl added this to the Nextcloud 16 milestone Feb 19, 2019
@juliushaertl juliushaertl added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Feb 19, 2019
@juliushaertl
Copy link
Member Author

juliushaertl commented Feb 19, 2019

  • Acceptance tests fail

@jancborchardt
Copy link
Member

Don’t you think it’s useful to open the menu (or at least for the first link of a file)? It is only shown when relevant, when you create the link. People who are not familiar with Nextcloud (= most people) will discover the settings directly then.

@juliushaertl
Copy link
Member Author

Don’t you think it’s useful to open the menu (or at least for the first link of a file)? It is only shown when relevant, when you create the link. People who are not familiar with Nextcloud (= most people) will discover the settings directly then.

Hm thats a good point, on the other side if you just want to create a link and share it, opening the menu will actually require two clicks to get the link (one to close the menu and one to click the clipboard icon). Maybe we could just do that once for every user?

@jancborchardt
Copy link
Member

Maybe we could just do that once for every user?

Once for every user is fine too, if that’s possible. :)

@danxuliu
Copy link
Member

Acceptance tests fail

Sorry, I missed this; I will fix them... once you confirm that there are not going to be more changes to the behaviour ;-)

@juliushaertl
Copy link
Member Author

Sorry, I missed this; I will fix them... once you confirm that there are not going to be more changes to the behaviour ;-)

Thanks, I'll ping you if I need some help. 👍

@MorrisJobke MorrisJobke mentioned this pull request Jul 17, 2019
28 tasks
@juliushaertl
Copy link
Member Author

Obsolete with 18 and #276 which is done during the sidebar rewrite.

@juliushaertl juliushaertl deleted the bugfix/noid/link-share-menu branch July 18, 2019 07:06
@juliushaertl juliushaertl removed this from the Nextcloud 17 milestone Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress papercut Annoying recurring issue with possibly simple fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants