Skip to content
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

Add iOS shared color tokens (error, status, and presence) #2520

Merged
merged 5 commits into from
Jan 18, 2023

Conversation

lyzhan7
Copy link
Contributor

@lyzhan7 lyzhan7 commented Jan 13, 2023

Platforms Impacted

  • iOS
  • macOS
  • win32 (Office)
  • windows
  • android

Description of changes

Follow up to #2481

In the change above we added neutral and brand alias color tokens. There was still some ongoing discussion on shared color tokens (i.e. error, status, and presence tokens) at the time, but that's been resolved and those tokens have been added to the design-tokens repo. Updating iOS alias color tokens to include these new tokens.

After this change, the full set of the iOS alias tokens will be available.

Verification

Realized that the existing Tokens test page was actually getting updated with the iOS alias tokens changes - videos for light/dark mode below, the new shared colors that were added are at the bottom:

Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-01-13.at.10.39.46.mp4
Simulator.Screen.Recording.-.iPhone.14.Pro.-.2023-01-13.at.10.39.25.mp4

Pull request checklist

This PR has considered (when applicable):

  • Automated Tests
  • Documentation and examples
  • Keyboard Accessibility
  • Voiceover
  • Internationalization and Right-to-left Layouts

@lyzhan7 lyzhan7 requested a review from a team as a code owner January 13, 2023 18:51
@lyzhan7 lyzhan7 merged commit f7626f6 into microsoft:main Jan 18, 2023
@lyzhan7 lyzhan7 deleted the ios-shared-colors branch January 18, 2023 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants