Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Recipient field is misaligned in the Account modal #3560

Closed
elena-zh opened this issue Dec 27, 2023 · 5 comments
Closed

Recipient field is misaligned in the Account modal #3560

elena-zh opened this issue Dec 27, 2023 · 5 comments
Labels
Bug Something isn't working Design Design related Low Severity indicator for defects. It won't cause any major break-down of the system Regression

Comments

@elena-zh
Copy link
Contributor

Place a Swap order with a different recipient
Open Account modal and see the order details

AR: Recipient field is misaligned
image

ER: the field is properly aligned (the same intends, same alignment, same font)
@elena-zh elena-zh added Bug Something isn't working Design Design related Low Severity indicator for defects. It won't cause any major break-down of the system Regression labels Dec 27, 2023
@fairlighteth
Copy link
Contributor

@elena-zh Is this still actual? Because I've been testing with a different recipient address on Goerli and Gnosis Chain, but don't see the warning banner:
Screenshot 2024-01-15 at 17 49 27

@elena-zh
Copy link
Contributor Author

Hey @fairlighteth , I don't see that you are sending sell order to another recipient on your screenshot.
Please, add a recipient here
image
image

@fairlighteth
Copy link
Contributor

Seems like I had sent the wrong screenshot. Correct one:
Screenshot 2024-01-15 at 17 49 09

@fairlighteth
Copy link
Contributor

Screenshot 2024-01-16 at 10 36 35

Now on Goerli was able to re-produce

@elena-zh
Copy link
Contributor Author

When filled, it looks nice. When is open/creating/etc. - I see this issue.
image

@fairlighteth fairlighteth mentioned this issue Jan 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working Design Design related Low Severity indicator for defects. It won't cause any major break-down of the system Regression
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fairlighteth @elena-zh