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.

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

feat: refactor trade container styles #5103

Merged
merged 3 commits into from
Nov 19, 2024
Merged

Conversation

fairlighteth
Copy link
Contributor

Copy link

vercel bot commented Nov 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
cosmos ✅ Ready (Inspect) Visit Preview Nov 19, 2024 2:41pm
cowfi ✅ Ready (Inspect) Visit Preview Nov 19, 2024 2:41pm
explorer-dev ✅ Ready (Inspect) Visit Preview Nov 19, 2024 2:41pm
swap-dev ✅ Ready (Inspect) Visit Preview Nov 19, 2024 2:41pm
widget-configurator ✅ Ready (Inspect) Visit Preview Nov 19, 2024 2:41pm

@elena-zh
Copy link

Hey @fairlighteth , great!

1 question: should we also address this edge-case?
image
image

To reproduce:

  • pick a token pair that has a fee of 0.1% on Gnosis chain
  • fill in an amount to trade
  • fill in an enormous price

@fairlighteth
Copy link
Contributor Author

@elena-zh can you check again? I pushed an update.

@shoom3301 can't recall how I reproduced this error (basically playing around with large input amounts on the limit order):
Screenshot 2024-11-19 at 14 30 58

But it crashed the app.

Copy link

@elena-zh elena-zh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. As for the crash, I've opened #5110 issue as it is also reproducible on Prod.

@fairlighteth fairlighteth merged commit 29ab5c4 into develop Nov 19, 2024
11 of 12 checks passed
@fairlighteth fairlighteth deleted the overflow-text-fix branch November 19, 2024 16:33
@github-actions github-actions bot locked and limited conversation to collaborators Nov 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CSS broken on Limit Order
3 participants