<div class="gmail_quote">2011/4/1 David Edmundson <span dir="ltr"><<a href="mailto:david@davidedmundson.co.uk">david@davidedmundson.co.uk</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
In an effort to try and get this merged as soon as possible I've pushed a few fixes to your repo.</blockquote><div><br></div><div>Thanks.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><br><div>Shall I take on the QVariant craziness in the model (and fix all the double casts)</div><div><br></div><div>
Martin: </div><div>Can you take the sorting out refilter() issue, (remembering to add the fix into AccountsModelItem)</div><div>Also can you do the issue with icon loading as Dario explained then submit another diff.</div>
</div></blockquote><div><br></div><div>Yes, I will do that.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div>
<div><br></div><div>If you're going to be busy fixing up a new computer, let me know I'll sort out some of the bits and pieces.</div></div></blockquote><div><br></div><div>I'm all set up, but I can't compile tp-qt4 from git as there is some build error. But for the sake of time I'm downloading latest tarball release so I can get the rest compiled and set up. </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div><br></div><div>There's nothing stopping us fixing things after we've merged with the main repository.</div>
</div></blockquote><div><br></div><div>True. </div><div><br></div><div>Also, Dario, if there's anything left from the review comments, then go for it :)</div><div><br></div><div>Marty</div></div>