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

'copy to clipboard' button on datasets #10529

Merged
merged 9 commits into from
Nov 20, 2020

Conversation

OlegZharkov
Copy link
Contributor

@OlegZharkov OlegZharkov commented Oct 22, 2020

November 3rd is almost there. So why don't we do our own galaxy-elections? :)

if you are in favour of that button, please vote with 👍
image

This one is 🚀
image

Finally, this option is 🎉
image

Thanks, choose wisely

resolves #4073

@innovate-invent
Copy link
Contributor

innovate-invent commented Oct 22, 2020

Does this produce a popup that allows manually copying the link? Some browsers block pages from modifying the clipboard.
Also, my vote is #3

@OlegZharkov
Copy link
Contributor Author

OlegZharkov commented Oct 22, 2020

Does this produce a popup that allows manually copying the link? Some browsers block pages from modifying the clipboard.

If by 'some browsers' you mean IE we stopped supporting it a while ago. But if there's a real demand... sure, we can add 'manual copy' pop-up

there's compatibility chart at the end:
https://developer.mozilla.org/en-US/docs/Web/API/Navigator/clipboard

Also, my vote is #3

then put 🎉 emoji ;)

@innovate-invent
Copy link
Contributor

I use Firefox and have experienced issues with GitHubs 'clone url' copy feature. Perhaps it is just GitHub though.

@hexylena
Copy link
Member

Since we're bikeshedding: To me, the action is "Share". You're getting a link (chain), sure, but in order to send to someone else. So the Share icon makes the most sense in the context of what the user is trying to do.

@OlegZharkov
Copy link
Contributor Author

Since we're bikeshedding: To me, the action is "Share". You're getting a link (chain), sure, but in order to send to someone else. So the Share icon makes the most sense in the context of what the user is trying to do.

Well, to advocate for that, I also prefer https://fontawesome.com/icons/share, because we already have it in tools. Since it was there for years, galaxy users might have gotten used to this icon and the button functionality would be intuitive.
image

@nsoranzo
Copy link
Member

To me, the action is "Share". You're getting a link (chain), sure, but in order to send to someone else. So the Share icon makes the most sense in the context of what the user is trying to do.

Sure, but nowadays if I see a share icon (e.g. https://material.io/resources/icons/?icon=share&style=baseline ) on my phone, I expect to have a series of sharing option (various social networks, messaging apps, email) and probably also a copy to clipboard, which normally look like options 🚀 or 🎉 .
BTW, just noticed that Firefox on desktop also use the chain icon to "Copy Link" from the address bar.

Well, to advocate for that, I also prefer https://fontawesome.com/icons/share, because we already have it in tools. Since it was there for years, galaxy users might have gotten used to this icon and the button functionality would be intuitive.

I don't think the average user needs to open that menu very often, to be honest.

@natefoo
Copy link
Member

natefoo commented Oct 22, 2020

I didn't know the 👍 option was a share icon since on iOS it's different. Guess it's more like that on Android?

@hexylena
Copy link
Member

Sure, but nowadays if I see a share icon ... on my phone

Ah the fabled phone user of Galaxy :P I definitely have that expectation too, on the phone. But on the desktop I don't have that expectation, which I think is probably the bulk of users? I wish we had like, a group of biologists/end users we could poll on 5 minutes notice, sometimes.

I don't think the average user needs to open that menu very often, to be honest.

ahhh this is probably where I get my expectation within galaxy from, since I use that menu option super often.

@nsoranzo
Copy link
Member

I didn't know the +1 option was a share icon since on iOS it's different. Guess it's more like that on Android?

No, on Android it's the one I linked above: https://material.io/resources/icons/?icon=share&style=baseline

@martenson
Copy link
Member

I am fine with any as long as the popover says Copy link 🤣

@simonbray
Copy link
Member

Copied from gitter:

just to point out, dataset sharing has to be enabled in user preferences for this to work
I don't know if there is any good way to inform users of that. It took me a very long time to realize that option existed.

Also, I never clicked on that 'Share tool' icon once, which I suppose goes to show that it's easy to create nice features and hard to persuade users to use them. :)

@simonbray
Copy link
Member

FWIW an image search for 'copy link to clipboard' returns a lot of screenshots with chain and clipboard icons

@bgruening
Copy link
Member

I wish we had like, a group of biologists/end users we could poll on 5 minutes notice, sometimes.

One of our power users has just voted :)

@galaxybot galaxybot added this to the 21.01 milestone Oct 23, 2020
@OlegZharkov
Copy link
Contributor Author

OlegZharkov commented Oct 26, 2020

Copied from gitter:

just to point out, dataset sharing has to be enabled in user preferences for this to work
I don't know if there is any good way to inform users of that. It took me a very long time to realize that option existed.

thanks a lot, that's a valid point. If user restricted permissions only to himself, the button will be hidden.

Also we have a winner, 🎉 leads by triple digits! Thanks everyone for voting in this poll!

@nsoranzo
Copy link
Member

@OlegZharkov I think it makes sense to also use the chain icon and "Copy link" text for sharing a tool URL.

@OlegZharkov
Copy link
Contributor Author

@OlegZharkov I think it makes sense to also use the chain icon and "Copy link" text for sharing a tool URL.

right, will do

@simonbray
Copy link
Member

Copied from gitter:

just to point out, dataset sharing has to be enabled in user preferences for this to work
I don't know if there is any good way to inform users of that. It took me a very long time to realize that option existed.

thanks a lot, that's a valid point. If user restricted permissions only to himself, the button will be hidden.

Very cool, thanks a lot! And it looks like you even are doing it on the basis of the individual dataset's permissions, which is really nice.

@hexylena
Copy link
Member

thanks a lot, that's a valid point. If user restricted permissions only to himself, the button will be hidden.

isn't this more confusing for the user? That the button isn't always there for some users and they don't know why?

I know it would expand scope significantly but I'd really expect that the button exists always, and it guides the user through changing the permissions.

Or at least having it there but disabled with a note "your preferences do not permit sharing links to datasets" so they can know to go edit it.

@nsoranzo
Copy link
Member

Or at least having it there but disabled with a note "your preferences do not permit sharing links to datasets" so they can know to go edit it.

👍

@OlegZharkov
Copy link
Contributor Author

OlegZharkov commented Oct 30, 2020

Or at least having it there but disabled with a note "your preferences do not permit sharing links to datasets" so they can know to go edit it.

So, I did that:

image

However I am not sure, whether it's a good idea to show users a huge list of disabled buttons

@simonbray
Copy link
Member

But I am not sure, whether it's a good idea to show users a huge list of disabled buttons

I think it's ok, it also provides a nice visual way for the user to quickly see which datasets are shared. 👍

@dannon dannon merged commit 9984d23 into galaxyproject:dev Nov 20, 2020
@dannon
Copy link
Member

dannon commented Nov 20, 2020

Thanks @OlegZharkov

@OlegZharkov
Copy link
Contributor Author

Thanks @dannon and everyone who participated in this poll!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

offer 'copy to clipboard' button on datasets
10 participants