@fluentui-react-native/callout 0.19.1 → 0.19.5

package/CHANGELOG.json

@@ -2,7 +2,88 @@
   "name": "@fluentui-react-native/callout",
   "entries": [
     {
-      "date": "Thu, 16 Dec 2021 06:26:41 GMT",
+      "date": "Fri, 17 Dec 2021 22:04:25 GMT",
+      "tag": "@fluentui-react-native/callout_v0.19.5",
+      "version": "0.19.5",
+      "comments": {
+        "patch": [
+          {
+            "author": "beachball",
+            "package": "@fluentui-react-native/callout",
+            "comment": "Bump @fluentui-react-native/interactive-hooks to v0.11.4",
+            "commit": "b9ea5eb1409f37305169f75398fa7f41229cefbd"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Fri, 17 Dec 2021 19:53:21 GMT",
+      "tag": "@fluentui-react-native/callout_v0.19.4",
+      "version": "0.19.4",
+      "comments": {
+        "patch": [
+          {
+            "author": "beachball",
+            "package": "@fluentui-react-native/callout",
+            "comment": "Bump @fluentui-react-native/interactive-hooks to v0.11.3",
+            "commit": "3012e075ca89e36931357442679b73d20540af8b"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Fri, 17 Dec 2021 18:39:08 GMT",
+      "tag": "@fluentui-react-native/callout_v0.19.3",
+      "version": "0.19.3",
+      "comments": {
+        "patch": [
+          {
+            "author": "sanajmi@microsoft.com",
+            "package": "@fluentui-react-native/callout",
+            "commit": "b7787516d0c417fbe350793713ffce3330c0f4a7",
+            "comment": "Documentation fixes"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Fri, 17 Dec 2021 01:26:42 GMT",
+      "tag": "@fluentui-react-native/callout_v0.19.2",
+      "version": "0.19.2",
+      "comments": {
+        "patch": [
+          {
+            "author": "beachball",
+            "package": "@fluentui-react-native/callout",
+            "comment": "Bump @fluentui-react-native/interactive-hooks to v0.11.2",
+            "commit": "511aeb31982a5fb9dd2f7196c0c7217cc4a3dbc6"
+          },
+          {
+            "author": "beachball",
+            "package": "@fluentui-react-native/callout",
+            "comment": "Bump @uifabricshared/foundation-compose to v1.11.2",
+            "commit": "511aeb31982a5fb9dd2f7196c0c7217cc4a3dbc6"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Thu, 16 Dec 2021 23:00:56 GMT",
+      "tag": "@fluentui-react-native/callout_v0.19.1",
+      "version": "0.19.1",
+      "comments": {
+        "none": [
+          {
+            "author": "dannyvv@microsoft.com",
+            "package": "@fluentui-react-native/callout",
+            "commit": "25b3989739737bc598a5d699a941af4c0b4a421f",
+            "comment": "📦 applying package updates ***NO_CI***"
+          }
+        ]
+      }
+    },
+    {
+      "date": "Thu, 16 Dec 2021 19:10:30 GMT",
       "tag": "@fluentui-react-native/callout_v0.19.1",
       "version": "0.19.1",
       "comments": {
@@ -10,44 +91,44 @@
           {
             "author": "sanajmi@microsoft.com",
             "package": "@fluentui-react-native/callout",
-            "commit": "dd2f47a0a2d6aab7361301a16407c76497b45ce7",
-            "comment": "remove focus style overrides for macOS contextual menu items"
+            "commit": "51cbf7c64d5c52feeec2e845cd4dbd4e23c96e1e",
+            "comment": "Add up/down arrow key support to macOS ContextualMenu"
           },
           {
             "author": "sanajmi@microsoft.com",
             "package": "@fluentui-react-native/callout",
-            "commit": "e597717b14dce4916740076786391449dd4a7c69",
-            "comment": "Rename Jest snapshots to be platform agnostic "
+            "commit": "dd2f47a0a2d6aab7361301a16407c76497b45ce7",
+            "comment": "remove focus style overrides for macOS contextual menu items"
           },
           {
             "author": "sanajmi@microsoft.com",
             "package": "@fluentui-react-native/callout",
-            "commit": "51cbf7c64d5c52feeec2e845cd4dbd4e23c96e1e",
-            "comment": "Add up/down arrow key support to macOS ContextualMenu"
+            "commit": "e597717b14dce4916740076786391449dd4a7c69",
+            "comment": "Rename Jest snapshots to be platform agnostic "
           },
           {
             "author": "beachball",
             "package": "@fluentui-react-native/callout",
             "comment": "Bump @fluentui-react-native/adapters to v0.8.1",
-            "commit": "841d2998e8b97fabe0b6dac9c74c3d63d2b3bbd4"
+            "commit": "6a9f067ae2510481b6ff3512c10c08e7de84f86e"
           },
           {
             "author": "beachball",
             "package": "@fluentui-react-native/callout",
             "comment": "Bump @fluentui-react-native/interactive-hooks to v0.11.1",
-            "commit": "841d2998e8b97fabe0b6dac9c74c3d63d2b3bbd4"
+            "commit": "6a9f067ae2510481b6ff3512c10c08e7de84f86e"
           },
           {
             "author": "beachball",
             "package": "@fluentui-react-native/callout",
             "comment": "Bump @fluentui-react-native/tokens to v0.11.1",
-            "commit": "841d2998e8b97fabe0b6dac9c74c3d63d2b3bbd4"
+            "commit": "6a9f067ae2510481b6ff3512c10c08e7de84f86e"
           },
           {
             "author": "beachball",
             "package": "@fluentui-react-native/callout",
             "comment": "Bump @uifabricshared/foundation-compose to v1.11.1",
-            "commit": "841d2998e8b97fabe0b6dac9c74c3d63d2b3bbd4"
+            "commit": "6a9f067ae2510481b6ff3512c10c08e7de84f86e"
           }
         ]
       }

package/CHANGELOG.md

@@ -1,18 +1,51 @@
 # Change Log - @fluentui-react-native/callout
 
-This log was last generated on Thu, 16 Dec 2021 06:26:41 GMT and should not be manually modified.
+This log was last generated on Fri, 17 Dec 2021 22:04:25 GMT and should not be manually modified.
 
 <!-- Start content -->
 
+## 0.19.5
+
+Fri, 17 Dec 2021 22:04:25 GMT
+
+### Patches
+
+- Bump @fluentui-react-native/interactive-hooks to v0.11.4
+
+## 0.19.4
+
+Fri, 17 Dec 2021 19:53:21 GMT
+
+### Patches
+
+- Bump @fluentui-react-native/interactive-hooks to v0.11.3
+
+## 0.19.3
+
+Fri, 17 Dec 2021 18:39:08 GMT
+
+### Patches
+
+- Documentation fixes (sanajmi@microsoft.com)
+
+## 0.19.2
+
+Fri, 17 Dec 2021 01:26:42 GMT
+
+### Patches
+
+- Bump @fluentui-react-native/interactive-hooks to v0.11.2
+- Bump @uifabricshared/foundation-compose to v1.11.2
+
 ## 0.19.1
 
-Thu, 16 Dec 2021 06:26:41 GMT
+Thu, 16 Dec 2021 19:10:30 GMT
 
 ### Patches
 
+- Add up/down arrow key support to macOS ContextualMenu (sanajmi@microsoft.com)
 - remove focus style overrides for macOS contextual menu items (sanajmi@microsoft.com)
 - Rename Jest snapshots to be platform agnostic  (sanajmi@microsoft.com)
-- Add up/down arrow key support to macOS ContextualMenu (sanajmi@microsoft.com)
 - Bump @fluentui-react-native/adapters to v0.8.1
 - Bump @fluentui-react-native/interactive-hooks to v0.11.1
 - Bump @fluentui-react-native/tokens to v0.11.1

package/lib/Callout.types.d.ts

@@ -71,8 +71,8 @@ export interface ICalloutTokens extends IBackgroundColorTokens, CalloutBorderTok
 }
 export interface ICalloutProps extends IViewProps, ICalloutTokens {
     /**
-     * @platform win32
      * A string that should be announced when the callout is shown.
+     * @platform win32
      */
     accessibilityOnShowAnnouncement?: string;
     /**
@@ -80,10 +80,10 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      */
     componentRef?: React.RefObject<IFocusable>;
     /**
-     * @platform win32
      * Adds a beak to the Callout, pointing to the anchor target.
      * Notable Win32 limitation: Beak rendering currently limits the border width to its default, and the
      * border width prop will not be honored.
+     * @platform win32
      */
     isBeakVisible?: boolean;
     /**
@@ -91,7 +91,6 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      */
     onDismiss?: () => void;
     /**
-     * @platform win32
      * Callback invoked during callout dismissal; if set, focus will not be restored by the callout and onRestoreFocus must
      * result in focus being moved to the appropriate focusable target.
      *
@@ -100,6 +99,7 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      * this callback is most appropriate for components strictly controlling focus.
      *
      * restoreFocusEvent.nativeEvent.containsFocus is true if the Callout had focus while being dismissed.
+     * @platform win32
      */
     onRestoreFocus?: (restoreFocusEvent: RestoreFocusEvent) => void;
     /**

package/lib-commonjs/Callout.types.d.ts

@@ -71,8 +71,8 @@ export interface ICalloutTokens extends IBackgroundColorTokens, CalloutBorderTok
 }
 export interface ICalloutProps extends IViewProps, ICalloutTokens {
     /**
-     * @platform win32
      * A string that should be announced when the callout is shown.
+     * @platform win32
      */
     accessibilityOnShowAnnouncement?: string;
     /**
@@ -80,10 +80,10 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      */
     componentRef?: React.RefObject<IFocusable>;
     /**
-     * @platform win32
      * Adds a beak to the Callout, pointing to the anchor target.
      * Notable Win32 limitation: Beak rendering currently limits the border width to its default, and the
      * border width prop will not be honored.
+     * @platform win32
      */
     isBeakVisible?: boolean;
     /**
@@ -91,7 +91,6 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      */
     onDismiss?: () => void;
     /**
-     * @platform win32
      * Callback invoked during callout dismissal; if set, focus will not be restored by the callout and onRestoreFocus must
      * result in focus being moved to the appropriate focusable target.
      *
@@ -100,6 +99,7 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
      * this callback is most appropriate for components strictly controlling focus.
      *
      * restoreFocusEvent.nativeEvent.containsFocus is true if the Callout had focus while being dismissed.
+     * @platform win32
      */
     onRestoreFocus?: (restoreFocusEvent: RestoreFocusEvent) => void;
     /**

package/macos/Callout.swift

@@ -83,12 +83,8 @@ class CalloutView: RCTView, CalloutWindowLifeCycleDelegate {
 
 	// MARK: WindowLifeCycleDelegate
 
-	func didDetectHitOutsideCallout(calloutWindow: CalloutWindow) {
-		 dismissCallout()
-	}
-
-	func applicationDidResignActiveForCalloutWindow(calloutWindow: CalloutWindow) {
-		 dismissCallout()
+	func calloutWillDismiss(window: CalloutWindow) {
+		onDismissCallout()
 	}
 
 	// MARK: Private methods
@@ -96,12 +92,12 @@ class CalloutView: RCTView, CalloutWindowLifeCycleDelegate {
 	private func showCallout() {
 		updateCalloutFrameToAnchor()
 		calloutWindow.makeKeyAndOrderFront(self)
-		didShowCallout()
+		onShowCallout()
 	}
 
 	private func dismissCallout() {
 		calloutWindow.close()
-		didDismissCallout()
+		onDismissCallout()
 	}
 
 	/// Sets the frame of the Callout Window (in screen coordinates to be off of the Anchor on the preferred edge
@@ -268,11 +264,11 @@ class CalloutView: RCTView, CalloutWindowLifeCycleDelegate {
 		return calloutScreenRect
 	}
 
-	private func didShowCallout() {
+	private func onShowCallout() {
 		onShow?([:])
 	}
 
-	private func didDismissCallout() {
+	private func onDismissCallout() {
 		if let onDismiss = onDismiss {
 			guard let reactTag = reactTag else {
 				preconditionFailure("React Tag missing")
@@ -286,11 +282,11 @@ class CalloutView: RCTView, CalloutWindowLifeCycleDelegate {
 
 	private var anchorView: NSView?
 
-	private var proxyView: RCTView = RCTView()
+	private var proxyView = RCTView()
 
 	private lazy var calloutWindow: CalloutWindow = {
 		let window = CalloutWindow()
-		window.windowLifeCycleDelegate = self
+		window.lifeCycleDelegate = self
 
 		let visualEffect = NSVisualEffectView()
 		visualEffect.translatesAutoresizingMaskIntoConstraints = false

package/macos/CalloutWindow.swift

@@ -1,41 +1,38 @@
 import Foundation
 import AppKit
+import Carbon.HIToolbox
 
-protocol CalloutWindowLifeCycleDelegate {
-	/*
-	 * Notify the CalloutWindowLifeCycle object that the user touched/clicked outside the bounds of the callout.
-	 */
-	func didDetectHitOutsideCallout(calloutWindow: CalloutWindow)
-
-	/*
-	 * Notify the CalloutWindowLifeCycle object that the window the callout is in is no longer active.
-	 */
-	func applicationDidResignActiveForCalloutWindow(calloutWindow: CalloutWindow)
+protocol CalloutWindowLifeCycleDelegate: AnyObject {
+	/// Notify the delegate that the Callout is about to dismiss
+	func calloutWillDismiss(window: CalloutWindow)
 }
 
 class CalloutWindow: NSWindow {
 
-	public var windowLifeCycleDelegate: CalloutWindowLifeCycleDelegate?
+	weak var lifeCycleDelegate: CalloutWindowLifeCycleDelegate?
 
 	override init(contentRect: NSRect, styleMask style: NSWindow.StyleMask, backing backingStoreType: NSWindow.BackingStoreType, defer flag: Bool) {
 		super.init(contentRect: contentRect, styleMask: style, backing: backingStoreType, defer: flag)
-		isReleasedWhenClosed = true
+		isReleasedWhenClosed = false
 
 		styleMask = .borderless
 		level = .popUpMenu
 		backgroundColor = .clear
 		isMovable = false
 
-		NotificationCenter.default.addObserver(self, selector: #selector(appDidChangeActive(notification:)), name: NSApplication.didResignActiveNotification, object: nil)
+		// Dismiss the Callout if the window is no longer active.
+		NotificationCenter.default.addObserver(self, selector: #selector(dismissCallout), name: NSApplication.didResignActiveNotification, object: nil)
 
-		mouseEventMonitor = NSEvent.addLocalMonitorForEvents(matching: .leftMouseUp, handler: { (event) -> NSEvent? in
+		// Dismiss the Callout if the user touched/clicked outside the bounds of the callout.
+		mouseEventMonitor = NSEvent.addLocalMonitorForEvents(matching: .leftMouseUp, handler: { [weak self] (event) -> NSEvent? in
 			if (event.window != self) {
-				self.windowLifeCycleDelegate?.didDetectHitOutsideCallout(calloutWindow: self)
+				self?.dismissCallout()
 			}
 			return event
 		})
 	}
 
+	// Required to get a key view loop in the window
 	override var canBecomeKey: Bool {
 		return true
 	}
@@ -44,9 +41,20 @@ class CalloutWindow: NSWindow {
 		return false
 	}
 
-	@objc func appDidChangeActive(notification: NSNotification) {
-		windowLifeCycleDelegate?.applicationDidResignActiveForCalloutWindow(calloutWindow: self)
+	// Dismiss the Callout if the user presses the Escape Key
+	override func keyDown(with event: NSEvent)  {
+		switch Int(event.keyCode) {
+		case kVK_Escape:
+			dismissCallout()
+		default:
+			return super.keyDown(with: event)
+		}
+	}
+
+	@objc private func dismissCallout() {
+		lifeCycleDelegate?.calloutWillDismiss(window: self)
+		close()
 	}
 
 	private var mouseEventMonitor: Any?
-}
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fluentui-react-native/callout",
-  "version": "0.19.1",
+  "version": "0.19.5",
   "description": "A cross-platform Callout component using the Fluent Design System",
   "license": "MIT",
   "author": "",
@@ -22,10 +22,10 @@
   "dependencies": {
     "@fluentui-react-native/adapters": "^0.8.1",
     "@fluentui-react-native/component-cache": "^1.3.0",
-    "@fluentui-react-native/interactive-hooks": ">=0.11.1 <1.0.0",
+    "@fluentui-react-native/interactive-hooks": ">=0.11.4 <1.0.0",
     "@fluentui-react-native/tokens": ">=0.11.1 <1.0.0",
     "@uifabricshared/foundation-composable": ">=0.10.0 <1.0.0",
-    "@uifabricshared/foundation-compose": "^1.11.1",
+    "@uifabricshared/foundation-compose": "^1.11.2",
     "@uifabricshared/foundation-settings": ">=0.11.0 <1.0.0"
   },
   "devDependencies": {

package/src/Callout.types.ts

@@ -100,8 +100,8 @@ export interface ICalloutTokens extends IBackgroundColorTokens, CalloutBorderTok
 
 export interface ICalloutProps extends IViewProps, ICalloutTokens {
   /**
-   * @platform win32
    * A string that should be announced when the callout is shown.
+   * @platform win32
    */
   accessibilityOnShowAnnouncement?: string;
 
@@ -111,10 +111,10 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
   componentRef?: React.RefObject<IFocusable>;
 
   /**
-   * @platform win32
    * Adds a beak to the Callout, pointing to the anchor target.
    * Notable Win32 limitation: Beak rendering currently limits the border width to its default, and the
    * border width prop will not be honored.
+   * @platform win32
    */
   isBeakVisible?: boolean;
 
@@ -124,7 +124,6 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
   onDismiss?: () => void;
 
   /**
-   * @platform win32
    * Callback invoked during callout dismissal; if set, focus will not be restored by the callout and onRestoreFocus must
    * result in focus being moved to the appropriate focusable target.
    *
@@ -133,6 +132,7 @@ export interface ICalloutProps extends IViewProps, ICalloutTokens {
    * this callback is most appropriate for components strictly controlling focus.
    *
    * restoreFocusEvent.nativeEvent.containsFocus is true if the Callout had focus while being dismissed.
+   * @platform win32
    */
   onRestoreFocus?: (restoreFocusEvent: RestoreFocusEvent) => void;