Bug 155454 - Plasma doesn't remember desktop icons setting between restarts
Summary: Plasma doesn't remember desktop icons setting between restarts
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-11 12:32 UTC by Alex Merry
Modified: 2008-01-11 19:22 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Merry 2008-01-11 12:32:28 UTC
Version:           SVN r759781 (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

When I deselect the option "Show Desktop Icons" in the "Configure Desktop" dialog, the icons disappear as expected.  After restarting plasma (either logout and login, or kquitapp plasma; plasma) they are back, and the option is selected again.

After changing the setting, then running kquitapp plasma, neither plasmarc nor plasma-appletrc have a "DesktopIcons" group or a "showIcons" key.

I put this:

kDebug() << "Writing showIcons=" << (m_showIcons->checkState() == Qt::Checked ? true : false)
             << "to config group" << iconConfig.name()
             << "in config file" << iconConfig.config()->name();

At the end of BackgroundDialog::saveConfig(), and I get this line:

plasma(11484) BackgroundDialog::saveConfig: Writing showIcons= false to config group "DesktopIcons" in config file "plasmarc"

Placing iconConfig().config()->sync() at the end of BackgroundDialog::saveConfig() fixes the problem (the setting is saved and plasmarc gets a [AppletGlobals][plasma_containment_desktop][DesktopIcons] group with the correct keys), but this shouldn't be necessary, surely?
Comment 1 Aaron J. Seigo 2008-01-11 19:22:41 UTC
SVN commit 760003 by aseigo:

make globalConfig() work properly even for containments
BUG:155454


 M  +20 -5     applet.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=760003