Skip to content
This repository was archived by the owner on Feb 5, 2025. It is now read-only.

Conversation

@diegoreymendez
Copy link
Contributor

@diegoreymendez diegoreymendez commented Apr 27, 2021

Fixes a styling issue within the navigation bar (when embedded in a LoginViewController), that would show the wrong color for text in nav bar buttons and titles.

Related to wordpress-mobile/WordPress-iOS#16380

Known issues:

These will be tackled incrementally, to avoid holding this fix back.

  • The back button is not perfectly aligned with the left chevron.
  • The "Change Username" screen is shown twice the first time the user taps to open it.
  • The UI looks a bit odd in terms of the coloring, both in the login epilogue and in the username selection screens.

Associated PRs

WPiOS: wordpress-mobile/WordPress-iOS#16389

Testing:

Follow the instructions in the associated WPiOS PR.

@diegoreymendez diegoreymendez self-assigned this Apr 27, 2021
@diegoreymendez diegoreymendez added the bug Something isn't working label Apr 27, 2021
@diegoreymendez diegoreymendez requested a review from frosty April 27, 2021 15:40
@diegoreymendez diegoreymendez marked this pull request as ready for review April 27, 2021 15:42
Copy link
Contributor

@frosty frosty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can confirm navigation bar styles look correct in both light and dark mode 👍

@diegoreymendez diegoreymendez merged commit af93c64 into develop Apr 29, 2021
@diegoreymendez diegoreymendez deleted the try/fix-nav-bar-appearance-issue branch April 29, 2021 08:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants