<br><br><div class="gmail_quote">On Wed, Jun 26, 2013 at 4:19 PM, Francesco Nwokeka <span dir="ltr"><<a href="mailto:francesco.nwokeka@gmail.com" target="_blank">francesco.nwokeka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">me sad :(<div><br></div><div>It was initially thought to have your fav. contact(s) on your plasma desktop to access them quickly without having to pass through the contact list.</div><div><br></div><div>@David: what's wrong with the code? Can you point me out the bug so I can take a look at it before you scrap the whole thing?</div>
<div><div class="h5">
<div class="gmail_extra"><br></div></div></div></div></blockquote><div>ContactWrapper has properties.. however instead of a notify signal on the properties it emits a changed signal that someone else catches which then invokes a function on the root object in QML that updates the contact... it's all an incredibly over complex roundabout way of doing it.</div>
<div><br></div><div>There are functions being called to set a property when another property changes, which are completely against the point of property bindings.</div><div><br></div><div>I didn't understand QML at the time I reviewed it. I don't think many of us did, it was all rather new. So we have QML written like someone would write C++. </div>
<div><br></div><div>Whilst I can fix it to work round each bug in turn, it's not maintainable (or tbh maintained)</div><div><br></div><div>It can stay in if it can pass a d_ed code review by 0.7.</div><div><br></div><div>
Sorry.</div><div><br></div><div>David</div></div>