Use the nearest monitor when the search query rect does not intersect any monitor...
authorDana Jansens <danakj@orodu.net>
Mon, 1 Oct 2012 00:29:45 +0000 (20:29 -0400)
committerDana Jansens <danakj@orodu.net>
Sun, 7 Oct 2012 01:56:56 +0000 (21:56 -0400)
commit01f62ded2fe289fe245f4f05b5825f4bdcbe1dc3
tree52d3acf0c1730714b248ff2122b22ae46c033835
parent9a5160b88bc92dbfe5cc7e222a576367cea2fc5b
Use the nearest monitor when the search query rect does not intersect any monitor (Fix bug 5500)

Previously we would try to find the primary monitor and use that when the search
was outside any monitor. However, if the primary monitor is chosen by the mouse
position and the mouse is not inside any monitor, we enter infinite recursion
trying to find the primary monitor.

The nearest monitor is a better metric anyhow, and this ensures
screen_find_monitor() is never recursive as it always returns a value without
depending on other screen.c methods.
openbox/geom.h
openbox/screen.c