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

more safety in swapping containments around

Browse Source 
svn path=/trunk/KDE/kdelibs/; revision=1200095
This commit is contained in:
Aaron J. Seigo 2010-11-23 22:18:15 +00:00
parent 8e5bcfc79c
commit 3e354c4e81
1 changed files with 17 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
view.cpp
View File

@ -244,14 +244,14 @@ void View::setContainment(Plasma::Containment *containment)
Containment *oldContainment = d->containment;
int screen = -1;
int desktop = -1;
if (oldContainment) {
if (oldContainment != containment) {
screen = d->containment->screen();
desktop = d->containment->desktop();
}
} else {
int screen = d->lastScreen;
int desktop = d->lastDesktop;
if (oldContainment && oldContainment != containment) {
screen = oldContainment->screen();
desktop = oldContainment->desktop();
}
if (scene() != containment->scene()) {
setScene(containment->scene());
}
@ -267,15 +267,20 @@ void View::setContainment(Plasma::Containment *containment)
d->lastScreen = screen;
d->lastDesktop = desktop;
//kDebug() << "set screen from setContainment due to old containment";
oldContainment->setScreen(-1, -1);
containment->setScreen(-1, -1);
containment->setScreen(screen, desktop);
if (oldContainment && oldContainment != containment) {
oldContainment->setScreen(-1, -1);
}
if (screen != containment->screen() || desktop != containment->desktop()) {
containment->setScreen(screen, desktop);
}
} else {
d->lastScreen = otherScreen;
d->lastDesktop = otherDesktop;
}
if (oldContainment && oldContainment != d->containment && otherScreen > -1) {
if (oldContainment && oldContainment != containment && otherScreen > -1 &&
(!containment || otherScreen != containment->screen() || otherDesktop != containment->desktop())) {
// assign the old containment the old screen/desktop
//kDebug() << "set screen from setContainment" << screen << otherScreen << desktop << otherDesktop;
oldContainment->setScreen(otherScreen, otherDesktop);