paypal / PayPal-iOS-SDK

Accept credit cards and PayPal in your iOS app
Other
973 stars 368 forks source link

UIAppearance #373

Closed plu closed 7 years ago

plu commented 8 years ago

Hi,

I've read this section in the README:

Specifically, if you are using UIAppearance to modify the appearance of any UI elements in your app, you should reverse those changes prior to presenting our viewcontroller, and set them again after dismissing the viewcontroller.

I find it a bit cumbersome that consumers of your SDK have to do this. The consumers shouldn't have to worry about this. Instead the SDK could make sure that it sets all colors and font sizes to be displayed in your desired way.

What do you think about this?

Thanks, plu

bluk commented 8 years ago

We can try looking into this. I believe the reason why this is done is partially because we wouldn't know all of the possible things to restore (only you would depending on the configuration you use).

plu commented 8 years ago

Hey @bluk,

thank you for coming back on this issue. I have created a sample project that includes a real world UIAppearance setup of the eBay Kleinanzeigen iOS app:

PayPalAppearance

In the screenshots you can see how our UIAppearance setup is affecting the PayPal-iOS-SDK:

vs.

iamabhiee commented 8 years ago

Any updates for this issue?

bluk commented 8 years ago

No real updates. Identifying all of the different possible UIAppearance changes made by a merchant app and then correctly "undo"-ing them and then "redo"ing them is difficult without lots of possible breaking behavior. I'll definitely keep this in mind if anyone has any ideas or if I run across anything, but it is probably safer for you to make the UIAppearance calls as appropriate.