This is really just for people working on the contact list.<div><br></div><div>I've been doing a few minor edits on it recently and there is a lot of code that really needs tidying up before it can be merged. I thought I would highlight these here so we can all work on that, and get it fixed soon.</div>
<div><br></div><div>1) Coding Style</div><div><br>We follow this</div><div><a href="http://techbase.kde.org/Policies/Kdelibs_Coding_Style" target="_blank">http://techbase.kde.org/Policies/Kdelibs_Coding_Style</a></div>
<div><br></div><div>In particular </div><div>if (something) { <- brace on the end of the line. Always put the brace, even if it's for 1 line.</div><div><br></div><div>}</div><div><br></div><div>--------</div><div>
<br>
</div><div>2) Don't hardcode fonts and colours in the delegate code.</div><div><br></div><div>If you write font.setSize(10); that doesn't work if someone has vision issues or even just owns a massive resolution monitor. </div>
<div><br></div><div>Use KGlobalSettings::smallestReadableFont() </div><div> - and its equivalents for other fonts.</div><div><br></div><div>With this sizeHint code needs to be more complex, and use font metric information to work out the minimum height the box needs to be.</div>
<div><br></div><div>Same for colours. If you switch your colour scheme in systemsettings to "Obsidian Coast" a dark colour scheme, your delegate with the hardcoded background colour stands out really obviously as being wrong.</div>
<div>QPalette should have everything you need.</div><div><br></div><div>---------</div><div><br></div><div>3) Looking at whether the alias is blank or not seems a slightly hacky way to try and separate headings in the model from contacts.</div>
<div><br></div><div>I'm not sure what a better approach is, but there must be one.</div><div><br></div><div>--------</div><div><br></div><div>Overall it's still pretty awesome, hopefully this will speed up the process of having it merged into the real master.</div>
<div><br></div><div>I'm happy to help with any of these.</div><div><br></div><div>Dave</div>