Fix an issue with certain tooltips being placed on the wrong screen i…

…n certain multi screen configurations This did affect users with a vertical multi monitor setup using the classical (single Space) multi monitor mode.
sequelpro · Mar 25, 2015 · 34383efe75a6f655522c9cb6385b3cb97cd99e3c · 34383ef
1 parent 9fd78af
commit 34383efe75a6f655522c9cb6385b3cb97cd99e3c
Unified Split

Showing with 5 additions and 2 deletions.

+5 −2 Source/SPTooltip.m
diff --git a/Source/SPTooltip.m b/Source/SPTooltip.m
@@ -320,8 +320,11 @@ - (void)sizeToContent
 	NSScreen* candidate;
 	for(candidate in [NSScreen screens])
 	{
-		if(NSMinX([candidate frame]) < pos.x && NSMinX([candidate frame]) > NSMinX(screenFrame))
-			screenFrame = [candidate frame];
+		NSRect cf = [candidate frame];
+		if(NSPointInRect(pos,cf)) {
+			screenFrame = cf;
+			break;
+		}
 	}

 	// is contentView a webView calculate actual rendered size via JavaScript