<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/113824/">http://git.reviewboard.kde.org/r/113824/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 2b2b50566ca640fa5211d3f763b345be96ca6450 by Martin Klapetek on behalf of Alexandr Akulich to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On November 25th, 2013, 2:52 p.m. UTC, Alexandr Akulich wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy.</div>
<div>By Alexandr Akulich.</div>
<p style="color: grey;"><i>Updated Nov. 25, 2013, 2:52 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-desktop-applets
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">TelepathyContact:
Removed qmlObject related stuff.
m_contact renamed to m_contactWrapper as it should be.
Setup context property before loading qml file.
ContactWrapper:
Properties placed there them should be.
Added property isContactOnline;
Added notifier-signals information for properties.
Property canStartVideo renamed to canStartVideoCall (like canStartAudioCall).
undoAccountConnects() reimplemented to don't break connections possible maked outside.
Same about undoContactConnects().
Methods reordered in cpp file to group simular methods together.
Implemented updateProperties(). In future it's possible to implement granulated properties updating.
Added checks for isAccountOnline in presenceStatus().
Fixed coding-style.
QML part:
Get rid a lot of functions, that anyway doesn't work properly.
Get rid a lot of useless properties.
Don't check avatarPresenceStatus for empty string and don't replace it with default one, because it's already done in c++ part.
Get rid a lot of "anchors.fill: parent" and such, because it's bad practice to let children position itself on parent.
Comminication buttons reordered like in KDE IM Contacts application.
Side effects:
Fixed losted after few went offline/online presenceStatus.
Fixed communication buttons remaining available after account becomes offline.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Plasmoid state properly response on account and/or contact online/offline changing. Contact capabilities showed correctly.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>contact/src/contact-wrapper.h <span style="color: grey">(7b37907)</span></li>
<li>contact/src/contact-wrapper.cpp <span style="color: grey">(5ef5919)</span></li>
<li>contact/src/declarative/contents/ui/Avatar.qml <span style="color: grey">(6133563)</span></li>
<li>contact/src/declarative/contents/ui/Contact.qml <span style="color: grey">(ab434c8)</span></li>
<li>contact/src/declarative/contents/ui/DropDownMenu.qml <span style="color: grey">(7177c70)</span></li>
<li>contact/src/declarative/contents/ui/main.qml <span style="color: grey">(debb0ed)</span></li>
<li>contact/src/telepathy-contact.h <span style="color: grey">(fe95c83)</span></li>
<li>contact/src/telepathy-contact.cpp <span style="color: grey">(67c8a32)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113824/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>