alecs/plasma-framework
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

make sure there is any screen geometry returned

Browse Source 
if the heuristic fails (because the topleft of the dialog is offscreen)
use at least our screen()

the screen should be correctly updated now on Qt 5.3+ so should be
more reliable anyways (could be tried to remove the whole Q_FOREACH
at this point)

Change-Id: I29311f4767bde3e02ae7bcb54b400b72a8f190eb
This commit is contained in:
Marco Martin 2014-10-24 12:33:29 +02:00
parent 8d2aa05530
commit 2b07624173
1 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/plasmaquick/dialog.cpp
View File

@ -159,6 +159,17 @@ QRect DialogPrivate::availableScreenGeometryForPosition(const QPoint& pos) const
}
}
/*
* if the heuristic fails (because the topleft of the dialog is offscreen)
* use at least our screen()
* the screen should be correctly updated now on Qt 5.3+ so should be
* more reliable anyways (could be tried to remove the whole Q_FOREACH
* at this point)
*/
if (avail.isEmpty()) {
avail = q->screen()->availableGeometry();
}
return avail;
}