p2plabsxyz / dscan

A decentralized storage and file-sharing tool that uploads content to IPFS and generates decentralized QR codes.
https://chromewebstore.google.com/detail/dscan-own-your-identity-o/idpfgkgogjjgklefnkjdpghkifbjenap
MIT License
36 stars 26 forks source link

[feat]: add small "copied" popup feature after clicking on "copy CID… #18

Closed atifcppprogrammer closed 2 years ago

atifcppprogrammer commented 2 years ago

The following is a preview of how the "copied" tooltip appears, when the extension is used via the Chrome web browser.

akhileshthite commented 2 years ago

@atifcppprogrammer few changes.

atifcppprogrammer commented 2 years ago

@atifcppprogrammer few changes.

@AkhileshThite, what changes are required ?

atifcppprogrammer commented 2 years ago

Hmmm @AkhileshThite, I dont see any comments there :disappointed:

atifcppprogrammer commented 2 years ago

Hmmm @AkhileshThite, I dont see any comments there disappointed

Screenshot 2022-03-13 at 8 06 15 PM

Thanks can see it now, not sure why it was not visible before.

atifcppprogrammer commented 2 years ago

Hi @AkhileshThite, how about now ? The black border doesn't quite work in my view, please let me know which of the following you prefer so i can push it, apart from the changes you requested above, i have only changed "Copied !" to "COPIED" to since all the other buttons are upper case as well Screenshot from 2022-03-13 20-40-26 Screenshot from 2022-03-13 20-44-47

akhileshthite commented 2 years ago

Hi @AkhileshThite, how about now ? The black border doesn't quite work in my view, please let me know which of the following you prefer so i can push it, apart from the changes you requested above, i have only changed "Copied !" to "COPIED" to since all the other buttons are upper case as well Screenshot from 2022-03-13 20-40-26 Screenshot from 2022-03-13 20-44-47

Go with the first grey one 👍

atifcppprogrammer commented 2 years ago

Hi @AkhileshThite made a few minor changes in addition to the changes requested, change the letter spacing for the tool tip to match that of the buttons "FILE" and "FOLDER"

akhileshthite commented 2 years ago

Hi @AkhileshThite made a few minor changes in addition to the changes requested, change the letter spacing for the tool tip to match that of the buttons "FILE" and "FOLDER"

Sure, I'll test and merge in few days.

akhileshthite commented 2 years ago

@atifcppprogrammer Good work! All the updates are live with v2.1 release 👉 https://chrome.google.com/webstore/detail/dscan-decentralized-qr-co/idpfgkgogjjgklefnkjdpghkifbjenap