<div dir="ltr"><div><div>Hi,<br></div> So what should be done regarding this patch should it be applied to digiKam git repo.Also since patch works and does not break anything what else has to be done in this bug.<br><br>
</div>Thanks<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Apr 27, 2013 at 12:00 AM, Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div class="im">
<table style="border:1px #c9c399 solid" bgcolor="#f9f3c9" cellpadding="8" width="100%">
<tbody><tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/108382/" target="_blank">http://git.reviewboard.kde.org/r/108382/</a>
</td>
</tr>
</tbody></table>
<br>
</div><div class="im"><blockquote style="margin-left:1em;border-left:2px solid #d0d0d0;padding-left:10px">
<p style="margin-top:0">On April 26th, 2013, 5:54 p.m. UTC, <b>Marcel Wiesweg</b> wrote:</p>
<blockquote style="margin-left:1em;border-left:2px solid #d0d0d0;padding-left:10px">
<pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">OK. Sorting is good, the "T" action sounds good.
Putting existing tags to the top and the creation of a new tag to the bottom is a decision based on personal preference. The current behavior favors creating new tags, while the patched behavior favors assigning existing tags by keyboard. Gilles, your opinion?
Some more questions:
- You remove emit currentCompletionTextChanged(current->data...) Why? Unused, causing a bug?
- Why setCurrentRow(0)?</pre>
</blockquote>
</blockquote>
</div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">>Putting existing tags to the top and the creation of a new tag to the bottom is a decision based on personal preference.
So in this case tags list is not shorted by alpha ?
> The current behavior favors creating new tags,
yes.
>while the patched behavior favors assigning existing tags by keyboard. Gilles, your opinion?
We can considerate this patch introducing a new feature.
Try to use Tags/New tag menu option from AlbumGUI, it only available when a tag is selected from tag tree-view on the left side. It doesn't work when tag is selected from tree-view on the right side.
I we want to be homogeneous, It will be judicious to add a keyboard shortcut to this action, and whole context management must be changed to enable it when focus is given to one of all tags tree-view... No ?
Gilles
</pre><span class="HOEnZb"><font color="#888888">
<br>
<p>- Gilles</p></font></span><div><div class="h5">
<br>
<p>On April 21st, 2013, 1:04 p.m. UTC, Markus Leuthold wrote:</p>
<table style="background-image:url('');background-repeat:repeat-x;border:1px black solid" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
<tbody><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 style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody><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>
</tbody></table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
<tbody><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>
</tbody></table>
<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </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" target="_blank">View Diff</a></p>
</td>
</tr>
</tbody></table>
</div></div></div>
</div>
<br>_______________________________________________<br>
Digikam-devel mailing list<br>
<a href="mailto:Digikam-devel@kde.org">Digikam-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/digikam-devel" target="_blank">https://mail.kde.org/mailman/listinfo/digikam-devel</a><br>
<br></blockquote></div><br></div>