Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Price out of market location #1311

Closed
anxolin opened this issue Aug 24, 2021 · 1 comment
Closed

Price out of market location #1311

anxolin opened this issue Aug 24, 2021 · 1 comment

Comments

@anxolin
Copy link
Contributor

anxolin commented Aug 24, 2021

I love this new check we have:
image

However, my problem is that i don't know if it refers to the order item or the bottom order.
Maybe we can make it more clear?

Maybe, we don't even need to do one per order, and is enough to make the price RED and potencially show a warning next to it? This way, we only show this message on the top of all orders.

image

@anxolin anxolin changed the title Price out of market Price out of market location Aug 24, 2021
@fairlighteth
Copy link
Contributor

I see your points and agree we can improve it. Now that most of the order sidebar is covered, I will focus iterating on this. I think your suggestion to make the price in a specific color + text to indicate what's going on. Will iterate.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants