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] Fix usages of arguments in showNotification and hide in MSFNotification #1087

Merged

Conversation

huwilkes
Copy link
Collaborator

@huwilkes huwilkes commented Jul 21, 2022

Platforms Impacted

  • iOS
  • macOS

Description of changes

When we updated the hide method, we simplified the logic for when we do completionForHide(), but we unfortunately simplified it too much and removed the case where we need to actually dismiss the notification without animations. We also stopped passing the anchorView and animated down to the call to show the new toast after dismissing the current toast.

Verification

For testing purposes, I set the dismiss button action to not be animated, and removed the auto dismiss we have in the demo controller.

Before After
Notification_Dismiss_Before Notification_Dismiss_After

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

@huwilkes huwilkes requested a review from a team as a code owner July 21, 2022 01:06
@huwilkes huwilkes enabled auto-merge (squash) July 21, 2022 01:07
@huwilkes huwilkes merged commit 977135f into microsoft:fluent2-tokens Jul 21, 2022
huwilkes added a commit that referenced this pull request Jul 21, 2022
… in MSFNotification (#1087)

* Fix hide when animated is false

* Pass anchorView and animated through showNotification
@huwilkes huwilkes deleted the huwilkes-Notification-Hide branch December 9, 2022 01:06
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.

None yet

3 participants