-
Notifications
You must be signed in to change notification settings - Fork 124
Conversation
@@ -255,7 +259,9 @@ - (void)showRelativeToRect:(CGRect)positioningRect ofView:(NSView *)positioningV | |||
if (strongSelf.behavior == RBLPopoverBehaviorTransient) { | |||
shouldClose = !mouseInPopoverWindow; | |||
} else { | |||
shouldClose = strongSelf.popoverWindow.parentWindow.isKeyWindow && NSPointInRect(NSEvent.mouseLocation, strongSelf.popoverWindow.parentWindow.frame) && !mouseInPopoverWindow; | |||
shouldClose = strongSelf.popoverWindow.parentWindow.isKeyWindow && NSPointInRect(NSEvent.mouseLocation, strongSelf.popoverWindow.parentWindow.frame) && !mouseInPopoverWindow && !self.parentWindowResignedKey; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/self/strongSelf
Oh yeah... 🚎 |
🍍 |
🆒 |
Improve semi-transient behaviour
I don't understand the intent of this change. 😕 From the docs, it sounds like clicking in the main window should closing the popover. This change seems to require 2 clicks. Can you help me understand what the intent was here? |
This was to match |
And then drop it into the repo so we can close out #17. |
I made a test app in #123. From my testing, it looks like this change breaks the behavior. Maybe there was an |
Possibly? If something doesn't match what |
This fixes the case of first-clicks closing semi-transient popovers.