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

Hold to swap button #5920

Merged
merged 13 commits into from
Jul 27, 2024
Merged

Conversation

greg-schrammel
Copy link
Contributor

@greg-schrammel greg-schrammel commented Jul 9, 2024

Fixes APP-1660
Fixes APP-1678

What changed (plus any additional context for devs)

Screen.Recording.2024-07-09.at.14.42.29.mov

Screen recordings / screenshots

What to test

Copy link

linear bot commented Jul 9, 2024

@greg-schrammel greg-schrammel force-pushed the gregs/app-1660-add-hold-to-swap-button branch from 5774c37 to 2fc0dcd Compare July 9, 2024 17:45
@greg-schrammel greg-schrammel force-pushed the gregs/app-1660-add-hold-to-swap-button branch from 2fc0dcd to d509cfd Compare July 9, 2024 17:45
Copy link
Member

@christianbaroni christianbaroni left a comment

Choose a reason for hiding this comment

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

Looks like shadows are missing from the button

Copy link

linear bot commented Jul 27, 2024

@christianbaroni christianbaroni merged commit 8d7dcf7 into develop Jul 27, 2024
6 checks passed
@christianbaroni christianbaroni deleted the gregs/app-1660-add-hold-to-swap-button branch July 27, 2024 21:23
BrodyHughes added a commit that referenced this pull request Aug 1, 2024
…inbow into brody/enable-android-tests

* 'brody/enable-android-tests' of github.com:rainbow-me/rainbow:
  degen mode feature flag (#5969)
  Fix Android dapp browser gestures (#5964)
  Add Degen Mode, swap settings panel (#5963)
  Update flipping logic in SwapInputsController (#5948)
  Hold to swap button (#5920)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants