-
Notifications
You must be signed in to change notification settings - Fork 111
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
[Brand Updates] Watch app #14803
[Brand Updates] Watch app #14803
Conversation
@@ -0,0 +1,3 @@ | |||
/// Disposable Class to find this `Bundle` at runtime | |||
/// | |||
internal class WooFoundationBundleClass {} |
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.
I extracted this class from the UIColor+ColorStudio because some of the constructors used in it are not available in watchOS.
ba79628
to
bcf8050
Compare
📲 You can test the changes from this Pull Request in WooCommerce iOS by scanning the QR code below to install the corresponding build.
|
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.
Tested on simulator watch series 10 and confirmed that the colors match the new branding.
404827c
into
feature/woo-2.0-brand-updates
Closes: woocommerce/woomobile-private#407
Description
This PR updates the Watch app to use the new color palette, to do so, I updated the WooFoundationWatch library to include the shared color palette in addition to the
ColorStudio
helpers.Steps to reproduce
Testing information
Confirm the UI matches the new brand.
Tested using Watch Series 9 simulator, WatchOS 11.0
Screenshots
RELEASE-NOTES.txt
if necessary.Reviewer (or Author, in the case of optional code reviews):
Please make sure these conditions are met before approving the PR, or request changes if the PR needs improvement: