<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/108382/">http://git.reviewboard.kde.org/r/108382/</a>
</td>
</tr>
</table>
<br />
<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 Digikam, Gilles Caulier and Marcel Wiesweg.</div>
<div>By Markus Leuthold.</div>
<p style="color: grey;"><i>Updated April 21, 2013, 1:04 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">The only issue mentioned was the missing documentation visible to the user, therefore I've added the new tagging functionality to "Tips of the day". The patch was also tested by Saurabh. Anything else missing?</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description (updated)</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;">Make tagging more accessible by keyboard
* Pressing "T" will focus the tagedit-box.
* The tag is applied by pressing enter. Pressing enter a second time will
focus the mainwindow and advance to the next image.
* The dropdown of the tagedit-box remembers already entered tags, the
order of the dropdown items are sorted such that already entered tags appear first.
* add tagging-by-keyboard to "Tips of the day"</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;">I successfully use this feature on a regular basis. Also tested with current HEAD, works fine.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>data/tips <span style="color: grey">(eef2262ef39fe77a65113f75be750122bda0f3fb)</span></li>
<li>digikam/main/digikamapp.cpp <span style="color: grey">(d7769e60d4181f95149774744fb93e711761c2ed)</span></li>
<li>digikam/main/digikamapp_p.h <span style="color: grey">(2d9f0c20f65a29db153c868cbfcea2e5574b1bdd)</span></li>
<li>digikam/tags/addtagscompletionbox.cpp <span style="color: grey">(266421c34fe6d939b094d0c0b1dea77065e024ee)</span></li>
<li>digikam/tags/addtagslineedit.h <span style="color: grey">(1eec38c090eaefd592e4cc5c4561fadf04b9de26)</span></li>
<li>digikam/tags/addtagslineedit.cpp <span style="color: grey">(abcf5b1f9d9a0340a9838b267faa2637f989bd96)</span></li>
<li>digikam/views/digikamview.h <span style="color: grey">(669eac958eeaa5aacb52f0a55d879175c4abbe34)</span></li>
<li>digikam/views/digikamview.cpp <span style="color: grey">(89ae95684cef96da14df1807ab821f04d274471a)</span></li>
<li>libs/database/albumdb.cpp <span style="color: grey">(8ca4fd0e0323fb7c8bae65f947d21b9d9ee6b50c)</span></li>
<li>libs/imageproperties/imagedescedittab.h <span style="color: grey">(0dc7e31af05bf95a6caf5ee4edde962bda7c0e7a)</span></li>
<li>libs/imageproperties/imagedescedittab.cpp <span style="color: grey">(b6ce9494852b4a624addefa7c1fb1196ff265e68)</span></li>
<li>libs/imageproperties/imagepropertiessidebar.h <span style="color: grey">(f6703339fc2e1a5762f8db898e7ad69dddab7868)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108382/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>