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

MOB-430 : support error states for TP/SL inputs #134

Merged

Conversation

mike-dydx
Copy link
Contributor

Links (dYdX Internal Use Only)

Linear Ticket: MOB-430 : support error states for TP/SL inputs
Figma Design: https://www.figma.com/file/mKevZOfE9nj6MZpiolKYW1/dYdX-%E2%80%BA-Mobile?type=design&node-id=5621-14141&mode=design&t=MeYotwfkGOffPavy-4


Description / Intuition

  • adds in-line alert component (success, error, warning)
  • trigger in-line alert component when gain/loss values are negative (note, this should be done instead with Abacus validation errors

Before/After Screenshots or Videos

Before After
Simulator.Screen.Recording.-.iPhone.15.-.2024-04-08.at.16.40.55.mp4

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring or Technical Debt
  • Documentation update
  • Other (please describe: )

Copy link

linear bot commented Apr 8, 2024

@mike-dydx mike-dydx requested a review from ruixhuang April 8, 2024 20:43
@mike-dydx mike-dydx force-pushed the mike/mob-428-add-custom-amount-to-price-triggers-screen branch from 1d78173 to 2ca282d Compare April 8, 2024 20:44
@mike-dydx mike-dydx enabled auto-merge (squash) April 8, 2024 20:45
@mike-dydx mike-dydx changed the title Mike/mob 428 add custom amount to price triggers screen MOB-430 : support error states for TP/SL inputs Apr 8, 2024
Copy link

linear bot commented Apr 8, 2024

@mike-dydx mike-dydx force-pushed the mike/mob-428-add-custom-amount-to-price-triggers-screen branch 2 times, most recently from 815da0b to f1e425c Compare April 9, 2024 23:12
@mike-dydx mike-dydx force-pushed the mike/mob-428-add-custom-amount-to-price-triggers-screen branch from f1e425c to 1c3c3c0 Compare April 10, 2024 01:17
@mike-dydx mike-dydx merged commit e54c229 into develop Apr 10, 2024
2 checks passed
@mike-dydx mike-dydx deleted the mike/mob-428-add-custom-amount-to-price-triggers-screen branch April 10, 2024 01:43
mike-dydx added a commit that referenced this pull request Aug 20, 2024
* some ui

* add label accessory to platform input, split out input views into view models

* update advanced font

* touch up UI

* clean up

* add in-line alert UI
mike-dydx added a commit that referenced this pull request Aug 21, 2024
* some ui

* add label accessory to platform input, split out input views into view models

* update advanced font

* touch up UI

* clean up

* add in-line alert UI
mike-dydx added a commit that referenced this pull request Aug 21, 2024
* some ui

* add label accessory to platform input, split out input views into view models

* update advanced font

* touch up UI

* clean up

* add in-line alert UI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants