braintree_android icon indicating copy to clipboard operation
braintree_android copied to clipboard
verified publisher icon braintree

UI interaction states

Open saperi22 opened this issue 1 month ago • 3 comments

Summary of changes

  • Add UI interactions to buttons
UI interactions

Checklist

  • [ ] Added a changelog entry
  • [ ] Relevant test coverage
  • [ ] Tested and confirmed payment flows affected by this change are functioning as expected

Authors

List GitHub usernames for everyone who contributed to this pull request.

saperi22 avatar Dec 05 '25 17:12 saperi22

These look really cool! Just looking at the story and the figma, it looks like there's a couple other colors and stuff for each color option for the PP button for hover and focus and stuff. Is that supposed to be in this PR as well?

saralvasquez avatar Dec 06 '25 00:12 saralvasquez

This looks great code wise! I spun it up to test it and it seems like the focus indicator doesn't have a space between it and the button. Is there a special way to test the hover or is it just tabbing to the button? I also noticed the button color didn't change when the button was clicked and the spinner replaced the logo

saralvasquez avatar Dec 11 '25 23:12 saralvasquez

I spun it up to test it and it seems like the focus indicator doesn't have a space between it and the button.

Great catch! Updated it.

Is there a special way to test the hover or is it just tabbing to the button?

I think hover might not show up on mobile devices.

I also noticed the button color didn't change when the button was clicked and the spinner replaced the logo

Talked offline. It exists, just resets to default state when button is no longer pressed.

saperi22 avatar Dec 12 '25 00:12 saperi22