roots-id / roots-react-native

Mock RootsWallet UI, for fast UI/UX iterations
Apache License 2.0
4 stars 5 forks source link

Review missing mocks #11

Closed 2byrds closed 1 year ago

2byrds commented 1 year ago

Our current roots-react-native mocks are a great start. We need to catch-up with the rest of the RootsWallet UI. Eventually roots-react-native will always be ahead of RootsWallet, as we will iterate all UI changes here first (after figma mocks).

@AlexAndrei98 has completed several mocks that show existing and future UI features/modals/pages. He will close his tickets in the other repos and post the links to those mocks on this ticket so that @HunnySajid can review them and ask questions, etc.

2byrds commented 1 year ago

@HunnySajid take a look at the animatedGifs and vids here to see some of the parts we are still missing (most of it should already look faimiliar) https://drive.google.com/drive/folders/1ah7o1HoPL05ien6Z30tAN9aKE0tgKST0?usp=sharing For instance we need:

Also you can take a look at our settings screen and import/export modal https://youtu.be/6y_d2rTyoY4

HunnySajid commented 1 year ago

@2byrds Sure, I will take a look today and reach out with feedback.

HunnySajid commented 1 year ago

@2byrds I went through the mockup, it looks great. A quick question -- are we going to follow the one with white/light theme or sticking with the old/black theme for now?

Here is the list:

I need some clarity on the following:

Let's create some issues regarding the missing screens mentioned above and we can finish these before moving forward with the next. What do you say?

2byrds commented 1 year ago

Sounds great! For now, when thinking about a credential being revoked, think of a driver's license. If the expiration date is in the past, the license is no longer valid. So think of it as a simple call that returns a boolean, 'is this valid?' yes/no. I'll create those tickets.

2byrds commented 1 year ago

@HunnySajid as for the entities involved. Because this is a self-sovereign identity (some call it Web5) there are three main roles:

In our app. The holder sees other holders, verifiers, issuers as contacts. The chat view is a log of all of the transactions between you and them. And the credentials are all of the credentials you have been given or sent (in which case you are acting as an issuer).

thats a good start for now. happy to discuss more on a call.

2byrds commented 1 year ago

Regarding white theme, @AlexAndrei98 had presented that to us. We haven't said yes/no. My initial comment was that dark themes rule nowadays. But we'd love your input too @HunnySajid. Colors aside, one thing I'm challenging us to do is avoid pages. To keep the UX contact/chat/credential based. previous wallets have many pages and we are trying to use richchat and workflows instead. We would love to discuss more soon. But not quite yet, we still need to get the current mock complete so we can start iterating changes that can be incorporated into RootsWallet easily