Light-mode polish #426
Light-mode polish #426
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
DannyDelott
force-pushed
the
danny-lightmode-polish
branch
from
fd541e1 to
477f435
Compare
| }, | ||
| ); | ||
| const className = classNames("daisy-card p-4 bg-base-200 border border-1", { | ||
| "shadow-[rgba(50,_50,_105,_0.15)_0px_2px_5px_0px,_rgba(0,_0,_0,_0.05)_0px_1px_1px_0px]": |
There was a problem hiding this comment.
can we use the default tailwind shadows here, ie shadow-md. I notice there's the slighest bit more bleed than with the custom but it still looks good. wdyt?
There was a problem hiding this comment.
Yep good call, I was playing around with some examples from https://manuarora.in/boxshadows, but shadow-md is almost identical to this like you said.
DannyDelott
force-pushed
the
danny-lightmode-polish
branch
from
477f435 to
57ac400
Compare
| if (token.symbol) { | ||
| return <span className="daisy-label-text">{token.symbol}</span>; | ||
| } | ||
| })()} |
There was a problem hiding this comment.
My max button isn't showing for shorts. Same for you?
There was a problem hiding this comment.
This is intentional. We don't show one for shorts.
|
I think if we are going B&W across the board, we should remove the purple buttons and labels on the position table. It feels out of place. Same with the red on short action buttons. |
Yep, will tackle that in the next PR since I'll be refactoring the tables, those buttons are going away in fact! |
2 participants
elevationprop to Well component