fix: coin input value being cut by bottom line balance #286
Conversation
); | ||
|
||
return ( | ||
<div className="flex items-center justify-end gap-2 mt-2 whitespace-nowrap"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since you created a coin-balance.css
you can move all these tailwind helper classes for a unique class using coinBalance
in the root div
component and maybe creating a coinBalance--balance
class also
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's better to remove coin-balance.css
and keep everything inline, no needs to have separate file to hold 1 class. will do that
.gitignore
Outdated
@@ -41,7 +41,8 @@ node_modules | |||
|
|||
# testing | |||
coverage | |||
cypress | |||
cypress/videos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cypress should cover both videos and screenshots
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes but only cypress
it's ignoring other cypress stuffs, like the e2e tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use path specfic /cypress
CoinBalance
component fromCoinSelector
to centralize balance behavior in a different line and fix the layout bug.Closes Coin input is getting cut when balance and typed value are too big #236