<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/105521/">http://git.reviewboard.kde.org/r/105521/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On November 20th, 2012, 10:52 a.m., <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Am I right in thinking Aleix fixed this properly?
I'm closing this if I don't hear back in a week.</pre>
</blockquote>
<p>On November 20th, 2012, 11:04 a.m., <b>Martin Klapetek</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I don't remember/know all the details, but scrollbars in qml are not fixed yet - and it looks impossible to do without "abusing" the cacheBuffer property because of the different delegate heights. But I'm happy to be proven wrong.</pre>
</blockquote>
<p>On November 20th, 2012, 12:01 p.m., <b>Aleix Pol Gonzalez</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">What I did was to make sure that the view stays at the bottom when the conversation changes.
See https://git.reviewboard.kde.org/r/106791/diff/#index_header
And AFAIK, it works fine...</pre>
</blockquote>
<p>On November 20th, 2012, 12:07 p.m., <b>David Edmundson</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">so that patch doesn't fix this, and this should still be merged?</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Well there's the test for ignoring joint words that doesn't look like it belongs to the patch, a modification to the delegate and the scrollbar that seem unrelated too.
The only thing that could be interesting is the cacheBuffer and I'm unsure if it's going to improve anything...</pre>
<br />
<p>- Aleix</p>
<br />
<p>On July 12th, 2012, 7:54 a.m., Lasath Fernando 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.</div>
<div>By Lasath Fernando.</div>
<p style="color: grey;"><i>Updated July 12, 2012, 7:54 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;">This fixes the annoying bug stopping the plasmoid from scrolling properly as elegantly as possible.
The problems is that because delegates are created on demand, there's no way for the ListView to determine the height of all its content. It currently just takes the size of one delegate, and multiplies it by the amount of items in the model. While that works in scenarios where the delegates are all the same height, it does not here.
What this does is -abuse- use the cacheBuffer property to force it to keep all delegates in ram. While it uses more memory, it let's us work around this issue (Only other option is to remove the scrollbar).
PS: I tried to set the cacheBuffer to view.count, but that made it ignore it completely (I think it can't change after initialized for it to be taken seriously).</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 to myself.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=293522">293522</a>
</div>
<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">(90cc07d)</span></li>
<li>plasmoid/org.kde.ktp-chatplasmoid/contents/ui/OutgoingDelegate.qml <span style="color: grey">(d713958)</span></li>
<li>tests/message-processor-basic-tests.h <span style="color: grey">(35f931f)</span></li>
<li>tests/message-processor-basic-tests.cpp <span style="color: grey">(554ce1a)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105521/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>