<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/110459/">http://git.reviewboard.kde.org/r/110459/</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;">Please, can somebody have a look at this patch?</pre>
<br />
<p>- José</p>
<br />
<p>On May 15th, 2013, 7:39 p.m. UTC, José Millán Soto 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 kde-workspace.</div>
<div>By José Millán Soto.</div>
<p style="color: grey;"><i>Updated May 15, 2013, 7:39 p.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 patch allows users to prevent important items from being overwritten in the clipboard history.
Methods keep and setKeep were added to HistoryItem. If an item has keep set to true, it will not be overwritten.
A submenu was added to set whether an item should be kept in history.
Even if there are more elements in the list than the maximum number of entries, the newest element copied to the clipboard will be in the clipboard item.
In order to allow Klipper to read history files written with previous versions of Klipper, saving and loading if an item should be kept is handled in klipper.cpp and not by the item.</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=54212">54212</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>klipper/history.cpp <span style="color: grey">(49e9bb0)</span></li>
<li>klipper/historyitem.h <span style="color: grey">(6c41d5d)</span></li>
<li>klipper/historyitem.cpp <span style="color: grey">(21fbe4e)</span></li>
<li>klipper/klipper.h <span style="color: grey">(bbfd9c9)</span></li>
<li>klipper/klipper.cpp <span style="color: grey">(cf41bd2)</span></li>
<li>klipper/klipperpopup.h <span style="color: grey">(31beff2)</span></li>
<li>klipper/klipperpopup.cpp <span style="color: grey">(bff2c25)</span></li>
<li>klipper/popupproxy.h <span style="color: grey">(910bd6e)</span></li>
<li>klipper/popupproxy.cpp <span style="color: grey">(555f383)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110459/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>