-
Notifications
You must be signed in to change notification settings - Fork 11
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
ULS: Update Help button color #327
Conversation
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.
as long as the conflict is resolved, !
@@ -24,6 +24,9 @@ class SiteCredentialsViewController: LoginViewController { | |||
override func viewDidLoad() { | |||
super.viewDidLoad() | |||
|
|||
navigationItem.title = WordPressAuthenticator.shared.displayStrings.logInTitle | |||
styleNavigationBar(forUnified: true) | |||
|
|||
localizePrimaryButton() |
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.
is it just my browser or GitHub is messing up with the indentation? When I open it in Xcode, everything is in order anyway.
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 noticed that too when Thuy first added this file. It appeared all kinds of wrong on the PR, but in code it's fine. 🤷 .
Thanks @Gio2018 ! |
Ref: #315
This adds functionality to set the
Help
nav button text color based on the nav bar being displayed. It also enables the unified nav bar on the Site Credentials view.For the unified flows, it is now the same blue as the
Back
button.For the original flows, it is still white.
To note, when tapping
Help
and theSupport
view is displayed, theClose
button (left bar button) color is incorrect for the unified flow. Stay tuned!Can be tested with WPiOS PR: wordpress-mobile/WordPress-iOS#14457