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

DAO and Fund section in Bisq seems to be missing ability to copy links #5278

Closed
pazza83 opened this issue Mar 6, 2021 · 11 comments · Fixed by #6314
Closed

DAO and Fund section in Bisq seems to be missing ability to copy links #5278

pazza83 opened this issue Mar 6, 2021 · 11 comments · Fixed by #6314

Comments

@pazza83
Copy link

pazza83 commented Mar 6, 2021

Description

A minor annoyance of mine but the whole DAO and Fund section in Bisq seems to be missing ability to copy links.

Links all have a little link icon. They all seem to be missing the little copy icon.

This is most annoying when doing a transaction and then providing the Tx ID. Currently I users have to visit the website and then copy the Tx ID from the URL or the webpage. It would be much easier if users could just press the little 'copy icon'. It would also reduce the risk of user less error,

Version

v1.5.9

Steps to reproduce

Go to the DAO section is Bisq (No option to copy links in multiple sections)
Go to the Funds section is Bisq (No option to copy links in multiple sections)

You can copy from places such as Trade ID details screen

Expected behaviour

Go to the DAO section is Bisq (option to copy links in multiple sections)
Go to the Funds section is Bisq (option to copy links in multiple sections)

Use the same icon as in the Trade ID details screen

Screenshots

funds - Copy
No option to copy Tx ID

bonds - Copy
No option to copy Tx Id or GitHub issue

@BtcContributor
Copy link
Contributor

I can take care of this.

@BtcContributor
Copy link
Contributor

Update: I had several testing to do for Bisq and other fixes but I already started with this.
It will be released ASAP.

@pazza83
Copy link
Author

pazza83 commented May 16, 2021

Hi @BtcContributor how are you progressing with this issue?

@BtcContributor
Copy link
Contributor

Hi @pazza83,
unfortunately this is stuck at this moment as I had other tasks to do.

Btw, I am honestly having some issues implementing the fix, so if this is high priority please assign it to someone else.

@pazza83
Copy link
Author

pazza83 commented Sep 23, 2021

Hi @BtcContributor thanks for letting me know.

@xyzmaker123 let me know if you would like to work on this issue.

@ghost
Copy link

ghost commented Sep 28, 2021

@pazza83 There is no "copy" button there, but links are clickable (after click browser with btc/bsq explorer is opened). Isn't it enough?

@pazza83
Copy link
Author

pazza83 commented Sep 28, 2021

Hi @xyzmaker123 the copy button is more secure, private and user friendly.

When trying to get transaction IDs you are forced to open a new window with a blockchain explorer and then copy the transaction ID from that site. This has some privacy concerns and creates a second point of error that you are copying the transaction id from a 3rd party website not from within your Bisq app.

@ripcurlx
Copy link
Contributor

Hi @xyzmaker123 the copy button is more secure, private and user friendly.

When trying to get transaction IDs you are forced to open a new window with a blockchain explorer and then copy the transaction ID from that site. This has some privacy concerns and creates a second point of error that you are copying the transaction id from a 3rd party website not from within your Bisq app.

But you are asked the first time you click on such link if you just want to copy them or open them within a browser.

@ghost
Copy link

ghost commented Sep 30, 2021

@ripcurlx Got it - just wanted to be sure everybody agree we want this feature.

@ripcurlx
Copy link
Contributor

ripcurlx commented Oct 4, 2021

The problem I see with this feature request is the concrete implementation part. ATM there is the icon for the link, which is used everywhere in the client. When we want to introduce the copy icon, where should it be? Next to the link icon would be confusing IMO.

@pazza83
Copy link
Author

pazza83 commented Oct 17, 2021

Hi @ripcurlx

Maybe having the transaction IDs selectable / copy-able as per @jmacxx's solution here #5747 would work.

Main issue is trying to navigate the DAO > Wallet > Transaction information none of the text is able to be selected.

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

Successfully merging a pull request may close this issue.

4 participants