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