-
Notifications
You must be signed in to change notification settings - Fork 168
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
Update tokens to use UIKit
types, part 2
#1655
Update tokens to use UIKit
types, part 2
#1655
Conversation
FYI, I'm aware that I have some lingering |
ios/FluentUI.Demo/FluentUI.Demo/Demos/ActivityIndicatorDemoController.swift
Outdated
Show resolved
Hide resolved
ios/FluentUI.Demo/FluentUI.Demo/Demos/ButtonDemoController.swift
Outdated
Show resolved
Hide resolved
ios/FluentUI.Demo/FluentUI.Demo/Demos/ObjectiveCDemoColorProviding.m
Outdated
Show resolved
Hide resolved
I guess this change will make a future of maybe-bringing-to-Appkit one day not so easy? Ah, well, probably the right trade off :) |
ios/FluentUI.Demo/FluentUI.Demo/Demos/BottomSheetDemoController.swift
Outdated
Show resolved
Hide resolved
ios/FluentUI.Demo/FluentUI.Demo/Demos/DividerDemoController.swift
Outdated
Show resolved
Hide resolved
@Saadnajmi Maybe a little, but that's far enough off that I agree it's the right trade-off. And this pattern will be easier for AppKit consumers to use as well! |
Platforms Impacted
Description of changes
Three commits to bring our public token API fully to UIKit types.
ControlTokenSet
andGlobalTokens
, to use UIKit.GlobalTokens
, will temporarily host two APIs - a legacy one using existingColorValue
/DynamicColor
types, and a modern one that fully leverages UIKit types. This is mostly for high-traffic APIs that would be overly disruptive to change now.ControlTokenSet
, are being fully converted now. These are low-traffic APIs that would be difficult to maintain in two forms - fully converting now is the easiest path forward.FluentTheme
APIs, which have changed here. External clients had no access to the WIP APIs, so will not experience this disruption.ControlTokenSet
customization for testing purposes. This is not yet the case for most of our clients, so the impact externally will be less severe.Binary change
Total increase: 117,672 bytes
Total decrease: -178,400 bytes
Full breakdown
Verification
Two approaches:
Visual Verification
UIKit-Tokens.mp4
Pull request checklist
This PR has considered:
Microsoft Reviewers: Open in CodeFlow