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

Fix delete on empty move to previous. #4706

Merged
merged 1 commit into from
Mar 15, 2022

Conversation

michelleb-stripe
Copy link
Contributor

@michelleb-stripe michelleb-stripe commented Mar 15, 2022

Summary

Fix a bug where deleting a single character from a text field caused it to move to the previous field.

Testing

  • Added tests
  • Modified tests
  • Manually verified

Screenshots

Before: https://user-images.githubusercontent.com/77996191/158421275-4d4837c5-053a-480e-b0ff-08c178fb3d26.mov
After: https://user-images.githubusercontent.com/77996191/158421102-9c954758-d880-4245-8ac9-020737cb5ede.mov

@github-actions
Copy link
Contributor

Diffuse output:

Copy link
Contributor

@jameswoo-stripe jameswoo-stripe left a comment

Choose a reason for hiding this comment

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

LGTM verified that the delete button doesn't jump text fields when its not empty

@michelleb-stripe michelleb-stripe merged commit 23a2a48 into michelleb/credit-card-compose Mar 15, 2022
@michelleb-stripe michelleb-stripe deleted the delete-button branch March 15, 2022 19:20
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.

3 participants