Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Prefer UINavigationBar.Appearance over Forms settings #14923

Merged
merged 5 commits into from
Jan 12, 2022
Merged

Conversation

jfversluis
Copy link
Member

Description of Change

The current implementation for iOS13+ overrides the settings that users might set through UINavigationBar.Appearance. This change should take those into account.

Issues Resolved

API Changes

None

Platforms Affected

  • iOS

Behavioral/Visual Changes

TBD

Before/After Screenshots

Not applicable

Testing Procedure

TBD

PR Checklist

  • Targets the correct branch
  • Tests are passing (or failures are unrelated)

Copy link
Contributor

@jsuarezruiz jsuarezruiz left a comment

Choose a reason for hiding this comment

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

Let me know when is ready for Review.

@jfversluis jfversluis removed the DO-NOT-MERGE-!!! 🛑 This is in progress and needs to be updated before it can be merged. label Jan 11, 2022
@jfversluis
Copy link
Member Author

@jsuarezruiz if you still want to review, this one is ready!

This should take the following order (only on iOS 13+, older versions stay intact):

  1. Color set on Forms element directly
  2. Color set through UINavigationBar.BackgroundColor
  3. Default (light/dark theme aware) color

@jsuarezruiz jsuarezruiz merged commit 5f3ed96 into 5.0.0 Jan 12, 2022
@jsuarezruiz jsuarezruiz deleted the fix-13291 branch January 12, 2022 12:16
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] UINavigationBar BackgroundColor can no longer be changed
2 participants