<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 />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit d13f94a8c5cf18a7c3c19c522fa484e5b7fb1bd9 by Markus Leuthold to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On April 21st, 2013, 1:04 p.m. UTC, Markus Leuthold 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 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;">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;">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> </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>