<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 />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On August 13th, 2014, 7:01 nachm. UTC, <b>Daker Pinheiro</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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="https://git.reviewboard.kde.org/r/119772/diff/1/?file=305156#file305156line107" style="color: black; font-weight: bold; text-decoration: underline;">applets/clipboard/contents/ui/clipboard.qml</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">106</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span> <span class="p">(</span><span class="nx">event</span><span class="p">.</span><span class="nx">key</span> <span class="o">==</span> <span class="nx">Qt</span><span class="p">.</span><span class="nx">Key_Backspace</span> <span class="o">&&</span> <span class="nx">filter</span><span class="p">.</span><span class="nx">text</span> <span class="o">==</span> <span class="s2">""</span><span class="p">)</span> <span class="p">{</span></pre></td>
</tr>
</tbody>
</table>
<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;">Why not adding the case Qt.Key_Backspace on Switch?</p></pre>
</blockquote>
</blockquote>
<pre style="margin-left: 1em; 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;">Because it should only ignore that when the filter text is empty, and fall-through conditionally is ugly imho.</p></pre>
<br />
<p>- Kai Uwe</p>
<br />
<p>On August 13th, 2014, 6:55 nachm. UTC, 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, 6:55 nachm.</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/Menu.qml <span style="color: grey">(8865e43)</span></li>
<li>applets/clipboard/contents/ui/clipboard.qml <span style="color: grey">(2235b1f)</span></li>
<li>applets/clipboard/contents/ui/ClipboardItemDelegate.qml <span style="color: grey">(aa1cf60)</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>