-
Notifications
You must be signed in to change notification settings - Fork 38
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
Provide ways to define PayLaterButton color from xml #258
Conversation
@saperi22 Do you have more context on why the button changed from extending |
@sarahkoop context here: https://github.com/paypal/paypal-android/pull/245/files#r1521937925 |
@@ -72,6 +75,14 @@ class PayLaterButton @JvmOverloads constructor( | |||
) | |||
} | |||
|
|||
private fun updateColorFrom(typedArray: TypedArray) { |
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.
Not to expand the scope of this PR, but does this same bug exist for the PayPalCreditButton
and PayPalButton
?
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.
No, those didn't have changes.
The reason this broke is because of the change in inheritance.
Summary of changes
Addresses a Live Issue reported by a merchant.
PayLaterButton
not honoringpaypal_color
attribute in xml.PayLaterButton
used to extend fromPayPalButton
, but now it extends fromPaymentButton
. I'm providing a new attributepaylater_color
to change the button color in xml.Checklist
Authors
Screenshots