Hey David. None of these are blockers. Just stuff I noticed.
* There is this strange ACKNOWLEDGEMENTS file * themeconfig.cpp ** The delegate seems to have a fixed size of 128x128. This might not be ideal for hidpi screens ** ThemeConfig::dump - Extra indendation * thememodel.cpp - KStandardDirs and KGlobal headers are not required * usersmodel.cpp - The coding style is quite different