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

MOBILE-675: Native UI for In-app Card payments #44

Merged
merged 7 commits into from
Mar 23, 2022
Merged

Conversation

PocketDaniel
Copy link
Contributor

No description provided.

Copy link
Member

@edgar-zigis edgar-zigis left a comment

Choose a reason for hiding this comment

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

  • UITextField design should be similar to Android. At the moment dark mode background colour of the text fields is still white. Corner radius differs as well, no visible border.
  • When text field is focused, keyboard lifts continue button on top of the text fields. Instead scrolling area should increase and button should be lifted max to the position Android does.
  • If continue bank payment modal is displayed and YES/no is clicked, keyboard does not automatically hide

Sources/Kevin/Core/Architecture/KevinViewModel.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Custom/KevinTextField.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Custom/KevinTextField.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Custom/KevinTextField.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Custom/KevinTextField.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Custom/KevinValidationResult.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Styles/KevinTheme.swift Outdated Show resolved Hide resolved
Sources/Kevin/Core/Styles/KevinTheme.swift Outdated Show resolved Hide resolved
@edgar-zigis edgar-zigis merged commit 84b3b4c into master Mar 23, 2022
@edgar-zigis edgar-zigis deleted the MOBILE-675 branch March 30, 2022 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants