Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

Identicon copied indicator #89

Open
karl-kallavus opened this issue Mar 13, 2023 · 7 comments
Open

Identicon copied indicator #89

karl-kallavus opened this issue Mar 13, 2023 · 7 comments

Comments

@karl-kallavus
Copy link
Contributor

karl-kallavus commented Mar 13, 2023

Indicator that address has been copied when identicon is pressed

@statictype
Copy link
Contributor

i'm thinking we should have a different notification for a copy event. something like a small tooltip next to the address that was copied.
the notifications that appear in the corner should be related to chain events.
what do you have in mind @Goranch3 ?

@Goranch3
Copy link

@statictype we have max 3 properties to indicate the address has been copied, cursor, address icon and the message.
see images below. I really like the temporary swap of the identicon to checkmark, but it needs to swap back. That might be a complication.
Screenshot 2023-03-13 at 13 57 23
Screenshot 2023-03-13 at 13 57 45

@Goranch3
Copy link

Goranch3 commented Mar 13, 2023

other than that, yes this notification should not be of the same importance as the on-chain events

@statictype
Copy link
Contributor

I really like the temporary swap of the identicon to checkmark, but it needs to swap back. That might be a complication.

I agree, it's a bit of a complication. We could just do the tooltip for now.

@Goranch3
Copy link

Goranch3 commented Mar 13, 2023

Tooltip and it would be nice if cursor changed (from the one with + to the one without +). You can see why it's odd to keep the same cursor if you look at polkadot.js and how this component behaves

@karl-kallavus
Copy link
Contributor Author

karl-kallavus commented Mar 13, 2023

Since we decided to split tooltips and notifications I updated current issue and created another one for on-chain events & notifications

@karl-kallavus karl-kallavus changed the title Notification component Identicon copied indicator Mar 13, 2023
@Goranch3
Copy link

Since we decided to split notifications I updated current issue and created another one for on-chain events & notifications

I wouldn't call tooltips notifications they are in a way but always related to the state of the component.

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

No branches or pull requests

3 participants