Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[NavigationBar] Fixing function call in willMove(toWindow:) #2104

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

DanielMorM
Copy link
Contributor

@DanielMorM DanielMorM commented Nov 4, 2024

Platforms Impacted

  • iOS
  • visionOS
  • macOS

Description of changes

#2102 Introduced a fix that is calling the wrong API.

update(with:) => updateColors(for:)

This PR has considered:

  • Light and Dark appearances
  • iOS supported versions (all major versions greater than or equal current target deployment version)
  • VoiceOver and Keyboard Accessibility
  • Internationalization and Right to Left layouts
  • Different resolutions (1x, 2x, 3x)
  • Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)
  • iPad Pointer interaction
  • SwiftUI consumption (validation or new demo scenarios needed)
  • Objective-C exposure (provide it only if needed)
Microsoft Reviewers: Open in CodeFlow

@DanielMorM DanielMorM requested a review from joannaquu November 4, 2024 18:19
@DanielMorM DanielMorM requested a review from a team as a code owner November 4, 2024 18:19
@DanielMorM DanielMorM enabled auto-merge (squash) November 4, 2024 18:28
@DanielMorM DanielMorM merged commit 71df35b into main Nov 4, 2024
7 checks passed
@DanielMorM DanielMorM deleted the user/damoral/navigationbar-fixing-willMove branch November 4, 2024 20:13
@joannaquu joannaquu mentioned this pull request Nov 21, 2024
12 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants