[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/4.5/kdebase/workspace/plasma/desktop/shell
From:       Chani Armitage <chanika () gmail ! com>
Date:       2010-11-27 18:40:11
Message-ID: 20101127184011.E8230AC8A2 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1201420 by chani:

partial backport of r1180339

exportLayout function needed for
CCBUG: 248386

sorry I didn't get this in sooner.
CCMAIL: mueller@kde.org


 M  +3 -16     activity.cpp  


--- branches/KDE/4.5/kdebase/workspace/plasma/desktop/shell/activity.cpp #1201419:1201420
@@ -246,26 +246,13 @@
 {
     const QString name = "activities/" + m_id;
     KConfig external(name, KConfig::SimpleConfig, "appdata");
-    foreach (const QString &group, external.groupList()) {
-        KConfigGroup cg(&external, group);
-        cg.deleteGroup();
-    }
 
-    //TODO: multi-screen saving/restoring, where each screen can be
-    // independently restored: put each screen's containments into a
-    // different group, e.g. [Screens][0][Containments], [Screens][1][Containments], etc
-    KConfigGroup dest(&external, "Containments");
-    KConfigGroup dummy;
-    foreach (Plasma::Containment *c, m_containments) {
-        c->save(dummy);
-        c->config().reparent(&dest);
-        c->destroy(false);
-    }
+    //apparently this magic turns a kconfig into a kconfiggroup
+    KConfigGroup group = external.group(QString());
+    m_corona->exportLayout(group, m_containments.values());
 
-    external.sync();
     m_containments.clear();
     emit closed();
-    //FIXME only destroy it if nothing went wrong
     //TODO save a thumbnail to a file too
 
     KActivityController controller;
[prev in list] [next in list] [prev in thread] [next in thread]