- load() is called implicitely when the kcm is shown.

- It's not necessary to add the collection each time load is called. Prevents
      the shortcuts from beeing shown 3 times.
    - the readSettings and writeSettings calls aren't necessary since the
      actionCollection contains only global shortcuts and these are saved by
      the kdedglobalaccel daemon (correct Andreas?).

CCMAIL: ahartmetz@gmail.com

svn path=/trunk/KDE/kdebase/workspace/; revision=815402
This commit is contained in:
Michael Jansen 2008-06-01 17:33:20 +00:00
parent 8854248626
commit 08a0d13288

View file

@ -78,8 +78,6 @@ LookingGlassEffectConfig::LookingGlassEffectConfig(QWidget* parent, const QVaria
a->setProperty("isConfigurationAction", true);
m_ui->editor->addCollection(m_actionCollection);
load();
}
LookingGlassEffectConfig::~LookingGlassEffectConfig()
@ -95,14 +93,8 @@ void LookingGlassEffectConfig::load()
KConfigGroup conf = EffectsHandler::effectConfig("LookingGlass");
// m_ui->editor->readSettings(&conf);
int radius = conf.readEntry("Radius", 200);
m_ui->radiusSpin->setValue(radius);
m_actionCollection->readSettings();
m_ui->editor->addCollection(m_actionCollection);
emit changed(false);
}
@ -115,7 +107,6 @@ void LookingGlassEffectConfig::save()
conf.writeEntry("Radius", m_ui->radiusSpin->value());
m_actionCollection->writeSettings();
m_ui->editor->save(); // undo() will restore to this state from now on
conf.sync();