-
Notifications
You must be signed in to change notification settings - Fork 719
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
Add message icon to rows that do contain a memo #2888
Comments
Thanks for opening this issue. |
I don't think that using different color is intuitive type of a UI, because you need to know about it, like to read the instruction "if a color is X then it means you have memo", while using some "now days" UI "industry standard" where the user can quickly tell, "oh there is some message inside" by looking at the icon. The demo I used was just quick grab from google pics, so it's not the icon I desire, but just to show that having "message indicator" style icon is easy to discover and understand, the user would likely click/tap on the icon, and normal row popup would occur with all the details and 90% chance he would look at the bottom to notice memo text and from now on the user intuitively discovered the fact that "icon means memo is there + tap on the row to see it" and having approach with different row color doesn't do that, it would be more like "why is this row a different color?" |
The current idea is, with the color it would make you re-read said transaction text which then would indicate by saying “received shielded memo” but yes an icon is still absolutely needed and necessary but the icons I had proposed fell semi flat from “generics”. It will still make enough for a user to understand and differentiate |
…roject#2888) * Bail out in few more places when blockchain is not synced yet * Apply review suggestion
…roject#2888) * Bail out in few more places when blockchain is not synced yet * Apply review suggestion
Since there is no room to display a memo, maybe you could add an icon next to address to indicate that the transfer includes memo
Here is my primitive drawing
The text was updated successfully, but these errors were encountered: