<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/104554/">http://git.reviewboard.kde.org/r/104554/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks good to me, I'm a big fan of simplifying code.
> Line edits and 4.8.0
Yeah, I'm not entirely sure what to do with this. Given that it currently /works/ with plasmaWidgets I think we will wait until 4.9, though part of me thinks it should be safe to change now. And we want the PlasmaComponents version so that we can give it focus easily.
>I'll leave it up to the maintainers to decide what to do with this.
I think the maintainer is going to be you at this rate.</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/104554/diff/1/?file=56370#file56370line90" style="color: black; font-weight: bold; text-decoration: underline;">plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ChatWidget.qml</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#f0f0f0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">89</font></th>
<td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> leftMargin: 5</pre></td>
<th bgcolor="#f0f0f0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">87</font></th>
<td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> leftMargin: 5</pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Not really a fan of 'magic numbers'.
Ideally we should take margin from the parent's SVGitem, but I don't think that's doable.
Is there any other slightly neater way to do it?</pre>
</div>
<br />
<p>- David</p>
<br />
<p>On April 12th, 2012, 1:37 a.m., Aleix Pol Gonzalez wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Telepathy and David Edmundson.</div>
<div>By Aleix Pol Gonzalez.</div>
<p style="color: grey;"><i>Updated April 12, 2012, 1:37 a.m.</i></p>
<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;">Tries to address a few problems the chat view has and simplifies the code (a tiny bit).
- just reduce opacity when the view is at the end. Having a scroll flickering doesn't feel good and helps reduce the stress the scrollbar adds in such a small UI when it's not needed. I set the opacity to 0.3 but maybe 0 would be better, or not. Don't know.
- the chat has two behaviors: a browse mode where the user is looking into the log and doesn't want to be distracted and the user is looking into the newer things on the chat. This patch tries to improve the detection of the current state by making it more relaxed. This way it won't happen that if you write too fast, then everything's messed up. [1]
Other than that, I tried to port the PlasmaWidgets.LineEdit -> PlasmaComponents.TextField. It's not really possible because at 4.8 we can't know when the user pressed "enter". I guess you want to release 0.4 to work on 4.8 so this is not possible, I think. Maybe it should be backported?
[1] I'm unsure why we need these both having the screen version, I'll leave it up to the maintainers to decide what to do with this.</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;">talked with myself for a while, once again. a lot of "asd" (and permutations) where written tonight.</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>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/ChatWidget.qml <span style="color: grey">(740cb34)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104554/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>