<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/119772/">https://git.reviewboard.kde.org/r/119772/</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Ship It!</p></pre>
<br />
<p>- Martin Gräßlin</p>
<br />
<p>On August 13th, 2014, 9:29 p.m. CEST, Kai Uwe Broulik wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for Plasma and Martin Gräßlin.</div>
<div>By Kai Uwe Broulik.</div>
<p style="color: grey;"><i>Updated Aug. 13, 2014, 9:29 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-workspace
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">This patch simplifies the new Klipper plasmoid by removing the custom highlight implementation and using ListView's highlight property similar to Kickoff. This also gets rid of the annoying lagging-behind selection rectangle.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I also implemented keyboard navigation similar to Kickoff. Open the plasmoid and start typing and your input will be forwarded to the filter input. Use arrow up and down to navigate between entries and Return to select them. Not yet reachable via keyboard only are the action keys (arrow left/right conflicts with the input and tab also seems wrong. Maybe Shift+Return for run, Ctrl+B for Barcode, Ctrl+Return for Edit (or Ctrl+E) and Ctrl+Del for remove? No idea.</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Opening Klipper and typing and navigating with arrow keys as well as selecting items works fine. I also didn't find any major regressions from the new highlight.<br style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;" />
ListView does not handle varying delegate sizes well, unfortunately. We should limit the size of the delegate to max. the popup's height because when copying large blocks of code that thing gets almost unusable. Also, setting a large cacheBuffer could minimize the jumping around while scrolling but might come with a performance cost when opening the plasmoid as the delegate items need to be created.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">(I don't know what's wrong with the search input but all my input fields (KRunner, Widget explorer, ..) are broken atm)</p></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>applets/clipboard/contents/ui/ClipboardItemDelegate.qml <span style="color: grey">(aa1cf60)</span></li>
<li>applets/clipboard/contents/ui/Menu.qml <span style="color: grey">(8865e43)</span></li>
<li>applets/clipboard/contents/ui/clipboard.qml <span style="color: grey">(2235b1f)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/119772/diff/" style="margin-left: 3em;">View Diff</a></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">File Attachments </h1>
<li><a href="https://git.reviewboard.kde.org/media/uploaded/files/2014/08/13/0cfbbf3c-e221-4ce0-9a9c-0c7d6e25f642__newklipper.png">Looks like before</a></li>
</ul>
</td>
</tr>
</table>
</div>
</body>
</html>