| mart | |
| davidedmundson |
| Plasma |
This removes quite a lot of uneeded code and fixes the automatic default
handling from KConfigXT point of view (this was the wrong integration
scheme with combo boxes).
It also gives me a nice testing ground for a widget based KCM usable
with kcmshell. :-)
| Automatic diff as part of commit; lint not applicable. |
| Automatic diff as part of commit; unit tests not applicable. |
That's significantly tidier!
| kcms/qtquicksettings/renderer.kcfg | ||
|---|---|---|
| 10 | Will this mean we write "automaticloop" to the config file? If so, we'll need to change the KQuickAddons::QtQuickSettings::init | |
| kcms/qtquicksettings/renderer.kcfg | ||
|---|---|---|
| 10 | Nope. KConfigXT is so clever that if you write a value which is the default to it, it kills the key in the config file instead. :-) This is why you always want KConfigXT btw, managing defaults is error prone, people forget to kill the key when they do readEntry/writeEntry manually... which means you got something in the config file when the default changes, so you don't see the new default. | |