-
Notifications
You must be signed in to change notification settings - Fork 27
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
Fixed PaymentButton
sizing issue
#255
Conversation
Co-authored-by: Jax DesMarais-Leder <jdesmarais@paypal.com>
@@ -111,6 +111,7 @@ struct SwiftUIPaymentButtonDemo: View { | |||
size: selectedSize | |||
) | |||
.id(buttonID) | |||
.frame(maxWidth: .infinity) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are these needed? I removed them and tested in canvas and sim and seem to be okay without these changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
They're not needed but I have them there so that people, internally and externally, who aren't familiar with SwiftUI know that you can set the button widths. Pre-PR, this part was broken and you couldn't customize the width in SwiftUI. If it feels redundant and unnecessary, I can remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe better to remove them if they're not necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gotcha, I'll update but will leave one of the in
Reason for changes
This PR is copied from the original #247, which was branched from
next-major-version
and we concluded that this change is non-breaking and should go intomain
. So this PR is branched frommain
There was a sizing issue with the Payment Button when used in SwiftUI apps
Summary of changes
Checklist
Authors