| broulik | |
| ngraham | |
| mart |
| VDG | |
| Plasma |
The GTK KCM is now moved to Application Style KCM as a sub page. The
functionality is identical to the one, found in GTK KCM, as well for the
bugs :)
The page is only accessible, if the gtkconfig kded module is loaded,
because only with the module the configuration can be changed.
Depends on D26261
| No Linters Available |
| No Unit Test Coverage |
| Buildable 21057 | |
| Build 21075: arc lint + arc unit |
| kcms/style/gtkthemesmodel.cpp | ||
|---|---|---|
| 108 | this needs to be if (parent.isValid()) { Otherwise your're saying every item has children with N rows, which have children with N rows forever. | |
| kcms/style/gtkthemesmodel.cpp | ||
|---|---|---|
| 108 | Did you mean if (parent.isValid()) {
return 0;
}
return m_themesList.count();? | |
| kcms/style/gtkthemesmodel.cpp | ||
|---|---|---|
| 108 | I think so :D | |
| kcms/style/gtkthemesmodel.h | ||
|---|---|---|
| 37 | Whar's the purpose of this destructor? | |
| kcms/style/gtkthemesmodel.h | ||
|---|---|---|
| 37 | No purpose, therefore can I just delete it? | |
| kcms/style/gtkthemesmodel.h | ||
|---|---|---|
| 37 | Yes, I don't see why one would want ~Class() override = default in a header file. | |