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

[Fluent2 Tokens] Merge main to fluent2-tokens #1235

Conversation

mischreiber
Copy link
Contributor

@mischreiber mischreiber commented Sep 9, 2022

Platforms Impacted

  • iOS
  • macOS

Description of changes

Merging additional PRs from main to fluent2-tokens:

Verification

Sanity pass of affected components, mostly those already merged to main.

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 and others added 9 commits September 7, 2022 14:08
* Move `fluentTheme` from window to view

* Add logic to only refresh if view is a descendent of the newly themed view
Currently we always render the toast image as template. If the client team pass in image that is set "renderAsOriginal" respect that mode.
)

use FluentUI as a clang module in objc, we want to publish module.modulemap next to the fluentui-swift.h
* First set of controls without spacing tokens

* Complete the rest of the controls

* Restoring in-use values

* Move layout constants to TokenSets
…microsoft#1229)

* Decrease CommandBar item group spacing to 8. Change received approval from the Fluent Design team.
* Enable LargeContentViewer for the CommandBar
@mischreiber mischreiber requested a review from a team as a code owner September 9, 2022 23:14
@mischreiber mischreiber merged commit 5b7b214 into microsoft:fluent2-tokens Sep 9, 2022
@mischreiber mischreiber deleted the mischreiber/fluent2-tokens-merge branch September 10, 2022 18:24
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.

6 participants