Use frame instead of visibleRect (#3043)

rust-windowing · Sep 27, 2023 · 55f60a3 · 55f60a3
1 parent 640c51f
commit 55f60a3
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,8 @@ And please only add new entries to the top of this list, right below the `# Unre
 
 # Unreleased
 
+- Fix window size sometimes being invalid when resizing on macOS 14 Sonoma.
+
 # 0.28.6
 
 - On macOS, fixed memory leak when getting monitor handle.

diff --git a/src/platform_impl/macos/appkit/view.rs b/src/platform_impl/macos/appkit/view.rs
@@ -44,9 +44,6 @@ extern_methods!(
             unsafe { msg_send_id![self, inputContext] }
         }
 
-        #[sel(visibleRect)]
-        pub fn visibleRect(&self) -> NSRect;
-
         #[sel(hasMarkedText)]
         pub fn hasMarkedText(&self) -> bool;
 

diff --git a/src/platform_impl/macos/view.rs b/src/platform_impl/macos/view.rs
@@ -209,7 +209,7 @@ declare_class!(
                 self.removeTrackingRect(tracking_rect);
             }
 
-            let rect = self.visibleRect();
+            let rect = self.frame();
             let tracking_rect = self.add_tracking_rect(rect, false);
             self.state.tracking_rect = Some(tracking_rect);
         }
@@ -221,7 +221,7 @@ declare_class!(
                 self.removeTrackingRect(tracking_rect);
             }
 
-            let rect = self.visibleRect();
+            let rect = self.frame();
             let tracking_rect = self.add_tracking_rect(rect, false);
             self.state.tracking_rect = Some(tracking_rect);