-
Notifications
You must be signed in to change notification settings - Fork 66
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
Support changing the custom tabs toolbar and navigation colors #1628
Conversation
ui-core/src/main/java/com/adyen/checkout/ui/core/internal/util/CustomTabsLauncher.kt
Show resolved
Hide resolved
<attr name="adyenStrokeColor" format="color" /> | ||
</declare-styleable> | ||
|
||
<declare-styleable name="AdyenCheckoutCustomTabs"> |
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.
Isn't this name supposed to be the same as a custom view? And since we don't have a view, is it necessary to declare a styleable?
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.
You're right I missed that, can you check the latest commit?
7d9dc10
to
4b348a9
Compare
Quality Gate failedFailed conditions |
Description
Currently we directly use
colorPrimary
as the toolbar color for our custom tabs where we launch external redirects. This PR allows changing this color as well as 3 other customizable colors (toolbarColor
,secondaryToolbarColor
,navigationBarColor
andnavigationBarDividerColor
).To preserve consistency with custom styling across the library, we added an XML style that can be overridden and will be applied internally to the custom tabs. This also allows easily setting different light and dark theme colors.
Checklist
COAND-907