status-mobile icon indicating copy to clipboard operation
status-mobile copied to clipboard
verified publisher icon status-im

Details overlapping on Wallet Send screen

Open mariia-skrypnyk opened this issue 1 year ago • 6 comments

Bug Report

Expected behavior

https://www.figma.com/design/HKncH4wnDwZDAhc4AryK8Y/Wallet-for-Mobile?node-id=220-148036&t=qIeoFmxPkmCAMFt1-0

Screenshot 2024-05-15 at 14 51 59

Actual behavior

Screenshot 2024-05-15 at 14 48 54

Full page Screenshot 2024-05-15 at 13 33 13

Reproduction

  1. Install app
  2. Login to app as user with a balance
  3. Go to wallet
  4. Send some tokens to another user

Additional Information

  • Status version: nightly 15/05
  • Operating System: Android
  • Device: Samsung A34 (14.0)

mariia-skrypnyk avatar May 15 '24 12:05 mariia-skrypnyk

@J-Son89 we currently display the address instead of the user's name. Is displaying the user's name out of scope?

OmarBasem avatar May 17 '24 07:05 OmarBasem

Thanks @OmarBasem - it seems this page needs design input for smaller screens right (sorry if that's already mentioned)

I think we should have the page looking perfect for addresses, once that's all working we can add support for the account names, saved addresses and contacts etc. I would say this part is lower priority though.

J-Son89 avatar May 17 '24 07:05 J-Son89

Or is the designs 'recipient gets ...' because that is much easier and the address is already in two places on the page

J-Son89 avatar May 17 '24 07:05 J-Son89

@J-Son89 "Recipient gets ..." sounds good to me

OmarBasem avatar May 17 '24 08:05 OmarBasem

@J-Son89 looking at the designs again, I am wondering what's the point of that part. Even what the recipient gets is already mentioned. I think displaying the time and max fees would be enough in that block

OmarBasem avatar May 17 '24 08:05 OmarBasem

Yeah that's true, can you raise this with design. It will help with the smaller screen issues too

J-Son89 avatar May 17 '24 08:05 J-Son89