<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd">
<html><head><meta name="qrichtext" content="1" /><style type="text/css">
p, li { white-space: pre-wrap; }
</style></head><body style=" font-family:'Ubuntu'; font-size:9pt; font-weight:400; font-style:normal;">
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello Saurabh</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">The patch works fine with current HEAD from today (after rebasing), I just tested it. Pressing T focusses the editbox, the sorting works as intended with the patch. You probably forgot to rebase?</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">I've added a "tip of the day" to make the feature visible to the user.</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">best, Markus</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">On Tuesday 09 April 2013 02.14:37 Saurabh Patel wrote:<br /></p>
<p style=" margin-top:12px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello,<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> As Gilles pointed out in his previous reply,this patch needs to be tested on digikam 3 where a tableview branch would be merged.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">So I tested this patch on digikam 3.2.0 beta.This patch didn't work at all.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">First pressing T doesn't open Tags section from rightsidebar.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Then autocompletion of tags as suggested by Markus doesn't work.It remains same for digikam after applying patch the code.<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Eg:Say I added tag 'abcde' and then tag 'abcdef' so according to patch 'abcdef' should be seen above 'abcde' in autocompletion since it is more recent but 'abcde' appears first in autocompletion than 'abcdef'<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">I had tested this on digikam 2 where this patch worked as Markus thought but now in digikam 3.2.0 beta it doesn't work<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">So kindly look into it<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Thanks<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">On Tue, Apr 2, 2013 at 8:02 PM, Saurabh Patel <<a href="mailto:saurabhpatel7717@gmail.com"><span style=" text-decoration: underline; color:#0057ae;">saurabhpatel7717@gmail.com</span></a>> wrote:<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:45px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello,<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> I recently got digikam 3.2.0 beta installed and saw that a sort of tool tip similar to tip required by this patch is already there with only difference that instead of pressing enter twice it requires pressing enter and then page down to assign tag to the image so it's just change of shortcut key.Also auto-completion feature based on alphabetical and most recently added tags is implemented as done by this patch so what further is to be done with this bug or should it be closed.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">When I was assigned this bug it was already implemented and had only to be tested which I have tested.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Thanks<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:45px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:45px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">On Tue, Mar 12, 2013 at 10:48 AM, Saurabh Patel <<a href="mailto:saurabhpatel7717@gmail.com"><span style=" text-decoration: underline; color:#0057ae;">saurabhpatel7717@gmail.com</span></a>> wrote:<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:50px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello,<br /> I tried to add tip of day for my assigned tag in digikam-software-compilation/core/data/Tips.html then I build digikam again using bootstrap.local,make,make install and tried to see if tip I added was visible in tip of day after running digikam but I was not able to see the tip I added.<br />Plz tell whether I am testing it correctly or not and also let me know if that tip is to be added somewhere also which might be problem.<br /><br />Thanks<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:50px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">On Mon, Mar 11, 2013 at 11:46 AM, Gilles Caulier <<a href="mailto:caulier.gilles@gmail.com"><span style=" text-decoration: underline; color:#0057ae;">caulier.gilles@gmail.com</span></a>> wrote:<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">2013/3/6 Saurabh Patel <<a href="mailto:saurabhpatel7717@gmail.com"><span style=" text-decoration: underline; color:#0057ae;">saurabhpatel7717@gmail.com</span></a>><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:60px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Hello,<br /> Regarding shortcut menu being made visible to the user,Tip of the day would be a good idea rather than status bar because tip of the day are meant for providing tips which can make our work easy but not necessary to apply.<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">You can patch tips of day content of course...</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:60px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> Regarding moving to next image after pressing enter twice,it seems difficult to be made obvious.Maybe along with the tip for the shortcut we could append this feature in tip.<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">patch need to be tested on my computer.</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:60px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"> Thnx for clarifying regarding sorting.<br /> Also is there anything which has to be done to this patch other than testing.<br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">3.1.0 release will be done soon. After that, a tableview branch will be merged to git/master. Patch much be adapted to this branch to compile fine. </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" color:#888888;"><br /></span></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><span style=" color:#888888;">Gilles Caulier</span></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:55px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">_______________________________________________<br />Digikam-devel mailing list<br /><a href="mailto:Digikam-devel@kde.org"><span style=" text-decoration: underline; color:#0057ae;">Digikam-devel@kde.org</span></a><br /><a href="https://mail.kde.org/mailman/listinfo/digikam-devel"><span style=" text-decoration: underline; color:#0057ae;">https://mail.kde.org/mailman/listinfo/digikam-devel</span></a><br /><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p></body></html>