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

Add rounded corners to BottomSheet #7144

Merged
merged 2 commits into from
Aug 21, 2023

Conversation

tillh-stripe
Copy link
Collaborator

@tillh-stripe tillh-stripe commented Aug 11, 2023

Summary

This pull request adds rounded corners to BottomSheet for a more modern look. We use the rounded corners value as provided in PaymentSheet.Configuration to nicely fit in with the merchant’s desired look.

Motivation

UX polish.

Testing

  • Added tests
  • Modified tests
  • Manually verified

Screenshots

Before After
straight-corners rounded-corners

Changelog

@tillh-stripe tillh-stripe force-pushed the tillh/use-bottomsheet-for-ps branch 3 times, most recently from 4b1032f to 45946ae Compare August 11, 2023 17:10
@tillh-stripe tillh-stripe force-pushed the tillh/bottom-sheet-rounded-corners branch 3 times, most recently from 950f9f2 to d9acb47 Compare August 14, 2023 14:23
@tillh-stripe tillh-stripe force-pushed the tillh/use-bottomsheet-for-ps branch 3 times, most recently from 4510480 to c85c75b Compare August 14, 2023 23:09
@tillh-stripe tillh-stripe force-pushed the tillh/bottom-sheet-rounded-corners branch 2 times, most recently from b88c55b to 9010735 Compare August 15, 2023 22:27
Base automatically changed from tillh/use-bottomsheet-for-ps to master August 21, 2023 21:33
@tillh-stripe tillh-stripe dismissed jaynewstrom-stripe’s stale review August 21, 2023 21:33

The base branch was changed.

@tillh-stripe tillh-stripe force-pushed the tillh/bottom-sheet-rounded-corners branch from 9010735 to 1573b3c Compare August 21, 2023 21:40
@tillh-stripe tillh-stripe marked this pull request as ready for review August 21, 2023 22:29
@tillh-stripe tillh-stripe requested review from a team as code owners August 21, 2023 22:29
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.

🎉

@tillh-stripe tillh-stripe merged commit 909882f into master Aug 21, 2023
7 checks passed
@tillh-stripe tillh-stripe deleted the tillh/bottom-sheet-rounded-corners branch August 21, 2023 23:27
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.

None yet

4 participants