Merge branch 'screening' into 'master'

Editor: Fall back to the closest screen when necessary (#8354)

Closes #8354

See merge request OpenMW/openmw!4542

Author: psi29a

apps/opencs/view/doc/view.cpp

@@ -1164,23 +1164,30 @@ void CSVDoc::View::onRequestFocus(const std::string& id)
 QScreen* CSVDoc::View::getWidgetScreen(const QPoint& position)
 {
     QScreen* screen = QApplication::screenAt(position);
-    if (screen == nullptr)
-    {
-        QPoint clampedPosition = position;
+    if (screen)
+        return screen;
+
+    const QList<QScreen*> screens = QApplication::screens();
+    if (screens.isEmpty())
+        throw std::runtime_error("No screens available");
 
-        // If we failed to find the screen,
-        // clamp negative positions and try again
-        if (clampedPosition.x() <= 0)
-            clampedPosition.setX(0);
-        if (clampedPosition.y() <= 0)
-            clampedPosition.setY(0);
+    int closestDistance = std::numeric_limits<int>::max();
+    for (QScreen* candidate : screens)
+    {
+        const QRect geometry = candidate->geometry();
+        const int dx = position.x() - std::clamp(position.x(), geometry.left(), geometry.right());
+        const int dy = position.y() - std::clamp(position.y(), geometry.top(), geometry.bottom());
+        const int distance = dx * dx + dy * dy;
 
-        screen = QApplication::screenAt(clampedPosition);
+        if (distance < closestDistance)
+        {
+            closestDistance = distance;
+            screen = candidate;
+        }
     }
 
     if (screen == nullptr)
-        throw std::runtime_error(
-            Misc::StringUtils::format("Can not detect the screen for position [%d, %d]", position.x(), position.y()));
+        screen = screens.first();
 
     return screen;
 }