ignore non current tabbed and on different activities clients

BUG: 313379
REVIEW: 108932
FIXED-IN: 4.11
This commit is contained in:
Thomas Lübking 2013-01-22 20:09:42 +01:00
parent 694d1a83b3
commit 769955dfb8

View file

@ -148,6 +148,24 @@ void Placement::placeAtRandom(Client* c, const QRect& area, Policy /*next*/)
c->move(tx, ty); c->move(tx, ty);
} }
// TODO: one day, there'll be C++11 ...
static inline bool isIrrelevant(Client *client, Client *regarding, int desktop)
{
if (!client)
return true;
if (client == regarding)
return true;
if (!client->isCurrentTab())
return true;
if (!client->isShown(false))
return true;
if (!client->isOnDesktop(desktop))
return true;
if (!client->isOnCurrentActivity())
return true;
return false;
}
/*! /*!
Place the client \a c according to a really smart placement algorithm :-) Place the client \a c according to a really smart placement algorithm :-)
*/ */
@ -198,27 +216,23 @@ void Placement::placeSmart(Client* c, const QRect& area, Policy /*next*/)
ToplevelList::ConstIterator l; ToplevelList::ConstIterator l;
for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) { for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) {
Client *client = qobject_cast<Client*>(*l); Client *client = qobject_cast<Client*>(*l);
if (!client) { if (isIrrelevant(client, c, desktop)) {
continue; continue;
} }
if (client->isOnDesktop(desktop) && xl = client->x(); yt = client->y();
client->isShown(false) && client != c) { xr = xl + client->width(); yb = yt + client->height();
xl = client->x(); yt = client->y(); //if windows overlap, calc the overall overlapping
xr = xl + client->width(); yb = yt + client->height(); if ((cxl < xr) && (cxr > xl) &&
(cyt < yb) && (cyb > yt)) {
//if windows overlap, calc the overall overlapping xl = qMax(cxl, xl); xr = qMin(cxr, xr);
if ((cxl < xr) && (cxr > xl) && yt = qMax(cyt, yt); yb = qMin(cyb, yb);
(cyt < yb) && (cyb > yt)) { if (client->keepAbove())
xl = qMax(cxl, xl); xr = qMin(cxr, xr); overlap += 16 * (xr - xl) * (yb - yt);
yt = qMax(cyt, yt); yb = qMin(cyb, yb); else if (client->keepBelow() && !client->isDock()) // ignore KeepBelow windows
if (client->keepAbove()) overlap += 0; // for placement (see Client::belongsToLayer() for Dock)
overlap += 16 * (xr - xl) * (yb - yt); else
else if (client->keepBelow() && !client->isDock()) // ignore KeepBelow windows overlap += (xr - xl) * (yb - yt);
overlap += 0; // for placement (see Client::belongsToLayer() for Dock)
else
overlap += (xr - xl) * (yb - yt);
}
} }
} }
} }
@ -251,25 +265,21 @@ void Placement::placeSmart(Client* c, const QRect& area, Policy /*next*/)
ToplevelList::ConstIterator l; ToplevelList::ConstIterator l;
for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) { for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) {
Client *client = qobject_cast<Client*>(*l); Client *client = qobject_cast<Client*>(*l);
if (!client) { if (isIrrelevant(client, c, desktop)) {
continue; continue;
} }
if (client->isOnDesktop(desktop) && xl = client->x(); yt = client->y();
client->isShown(false) && client != c) { xr = xl + client->width(); yb = yt + client->height();
xl = client->x(); yt = client->y(); // if not enough room above or under the current tested client
xr = xl + client->width(); yb = yt + client->height(); // determine the first non-overlapped x position
if ((y < yb) && (yt < ch + y)) {
// if not enough room above or under the current tested client if ((xr > x) && (possible > xr)) possible = xr;
// determine the first non-overlapped x position
if ((y < yb) && (yt < ch + y)) {
if ((xr > x) && (possible > xr)) possible = xr; basket = xl - cw;
if ((basket > x) && (possible > basket)) possible = basket;
basket = xl - cw;
if ((basket > x) && (possible > basket)) possible = basket;
}
} }
} }
x = possible; x = possible;
@ -286,22 +296,19 @@ void Placement::placeSmart(Client* c, const QRect& area, Policy /*next*/)
ToplevelList::ConstIterator l; ToplevelList::ConstIterator l;
for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) { for (l = m_WorkspacePtr->stackingOrder().constBegin(); l != m_WorkspacePtr->stackingOrder().constEnd() ; ++l) {
Client *client = qobject_cast<Client*>(*l); Client *client = qobject_cast<Client*>(*l);
if (!client) { if (isIrrelevant(client, c, desktop)) {
continue; continue;
} }
if (client->isOnDesktop(desktop) &&
client != c && c->isShown(false)) {
xl = client->x(); yt = client->y(); xl = client->x(); yt = client->y();
xr = xl + client->width(); yb = yt + client->height(); xr = xl + client->width(); yb = yt + client->height();
// if not enough room to the left or right of the current tested client // if not enough room to the left or right of the current tested client
// determine the first non-overlapped y position // determine the first non-overlapped y position
if ((yb > y) && (possible > yb)) possible = yb; if ((yb > y) && (possible > yb)) possible = yb;
basket = yt - ch; basket = yt - ch;
if ((basket > y) && (possible > basket)) possible = basket; if ((basket > y) && (possible > basket)) possible = basket;
}
} }
y = possible; y = possible;
} }