blink-mobile icon indicating copy to clipboard operation
blink-mobile copied to clipboard

Published 20 hours ago • verified publisher icon GaloyMoney

Remove Eye icon, Change QR scan icon

Open designsats opened this issue 1 year ago • 4 comments

  1. Eye icon to hide balance is redundant image

  2. Scan QR icon should be changed everywhere applicable. Current icon is industry standard for "Show QR" not scan.

image

designsats avatar Nov 21 '23 14:11 designsats

@designsats While the eye button is redundant, I can see it's purpose. Took me quite a while to discover that tapping the portfolio balance did the same thing. I'd leave the icon there, or if you really want we could move it elsewhere? Faded and to the right of the chart icon in the top left maybe?

MaxwellDG avatar Nov 22 '23 00:11 MaxwellDG

@designsats While the eye button is redundant, I can see it's purpose. Took me quite a while to discover that tapping the portfolio balance did the same thing. I'd leave the icon there, or if you really want we could move it elsewhere? Faded and to the right of the chart icon in the top left maybe?

Yes - you are right, its not intuitive enough without the eye. Lets keep the icon.

designsats avatar Dec 08 '23 00:12 designsats

I need a new SVG to replace th existing QR code SVG: https://github.com/GaloyMoney/galoy-mobile/blob/main/app/assets/icons-redesign/qr-code.svg

nicolasburtey avatar Dec 09 '23 20:12 nicolasburtey

qr-scanner

designsats avatar Dec 09 '23 20:12 designsats