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

Rename SwiftUI Color token APIs #2025

Merged
merged 1 commit into from
May 17, 2024

Conversation

mischreiber
Copy link
Contributor

@mischreiber mischreiber commented May 17, 2024

Platforms Impacted

  • iOS
  • visionOS
  • macOS

Description of changes

PR #2014 introduced a conflicting set of APIs for color, with the hope that most callers Swift callers would auto-resolve to the correct one. We've gotten customer feedback that this is not the case, so to unblock in the short term we'll be renaming the SwiftUI API. Longer-term, we'll come up with better names for both, but this should get folks unblocked quickly.

Binary change

n/a, only a method name change

Verification

Build and basic run of demo app.

Pull request checklist

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

@mischreiber mischreiber requested a review from a team as a code owner May 17, 2024 21:28
@mischreiber mischreiber enabled auto-merge (squash) May 17, 2024 22:03
@mischreiber mischreiber merged commit 559f4a4 into microsoft:main May 17, 2024
7 checks passed
@mischreiber mischreiber deleted the SwiftUIColors branch May 17, 2024 22:32
@lukas-cap lukas-cap mentioned this pull request May 21, 2024
12 tasks
@joannaquu joannaquu mentioned this pull request May 23, 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