Design review for new release

[AGKDesigns]

Inbox and portfolio:
Inbox icon not correct (Swap to comment icon)

Reduce the size of the X icon on the inbox to 80%

The switches on the filters are invisible on which and hard to see on black

Reduce text size of the date 2pt

Remove the blank empty space when there is no call to action on a message.

Dialogue pops every time you open inbox (assume this is still work in progress)

Wallet selector

Remove the combined heading.

Change Combined to Total

Change the tracked and untracked watch address titles to
Watch Address
Watch Address (Hidden)

As discussed last week, now that it goes straight to combined view on portfolio, perhaps we can

Can't tap combined option. It should just dismiss the wallet selector and return to combined view.

Not sure we need delete as an option for watch address here? I think it might be better to have a manage watch address screen in the general settings for edit and delete. They are not so common actions, and probably should be hidden most of the time.
Maybe these options could just be "View details" "Hide from combined total" Convert to wallet"?

Make the title of portfolio tappable

Settings
When swapping wallets from settings, the page doesn't update to new wallet

Help page needs a little spacing between items. Lots of room on page but everything is bunched up in the corner. Add maybe 12pt more between each item and the top and left edge.

Auto copy to clipboard if you tap the referral link box. Instead of having to manually select the address and press copy. Just like it does when you tap the QR code on settings.

I think the general settings should be above security center. While it's not as important, the hierarchy feels wrong currently.

Update referral UI to new design. (TBD)

Add manage watch address to general, with options to add delete and edit watch addresses

Security center updates TBD.

Security center not yet updated to auto select wallet (TBD)

Dark Mode
Switches on inbox filter are too dark
Seems pretty good overall otherwise

Extra points:
Remove the text and button from the featured items. I don't think it's really needed. The image has the text and we can add a CTA in the images if it's really needed. but already sort of obvious anyway.

[saltyskip]

I think all options for watch address should be in that screen, they have no other purpose except viewing in portfolio, do not need dedicated settings

[AGKDesigns]

I disagree, there is difference between managing and interacting and while placing them all together may be easier, I don't think it would be more intuitive for the user.

[saltyskip]

Watch addresses have no place in the app except in that one screen, why would it be more intuitive to place settings related to them somewhere else?

Especially since there will only be maybe 1 or 2 options

[AGKDesigns]

Because regardless of where it is used/interacted in the app, generally management tools are grouped together in the settings.

Same as wallets, you manage them together in the settings.

If you asked someone to delete their watch address, their first thought won't be to open the wallet selector and goto the watch address there to delete it. It would be to head to the settings I believe.

[saltyskip]

Also think we should remove unlock functionality of wtach address entirely, doesnt make sense given the use case of a watch address

[AGKDesigns]

Yeah that’s cool I think.

…

On May 16, 2019, at 17:01, andrei ***@***.***> wrote: Also think we should remove unlock functionality of wtach address entirely — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#184?email_source=notifications&email_token=AD6JHKSYQ7ENRYC23H2VCDDPVUIGJA5CNFSM4HMMBCTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVRAENQ#issuecomment-492962358>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AD6JHKVLHX7EBQF74H33VGTPVUIGJANCNFSM4HMMBCTA>.