<br><div><span class="gmail_quote">2007/6/7, Jan Kundrát <<a href="mailto:jkt@gentoo.org">jkt@gentoo.org</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Hi folks, this is what I have in my local checkout and would like to apply:<br><br><br>145158-136374-retag-from-last-image.patch:<br>------------------------------------------<br><br>a) Change from "<p>Changes made to current item, really cancel?</p>" to

<br>"<p>Some items were changed. Do you really want to undo all recent edits<br>for each affected image?</p>"<br><br>b) Allow to copy tags from previously edited image to current one (works<br>only in annotation dialog)

<br><br></blockquote></div><br>OK, reviewed this one. Seems good to me, though I think it would be clearer to add a tooltip at least on the copyPrevious button :<br><br>Index: /home/baptiste/Perso/Projets/KDE-Extragear-Graphics/kphotoalbum/AnnotationDialog/Dialog.cpp

<br>===================================================================<br>--- /home/baptiste/Perso/Projets/KDE-Extragear-Graphics/kphotoalbum/AnnotationDialog/Dialog.cpp    (revision 676302)<br>+++ /home/baptiste/Perso/Projets/KDE-Extragear-Graphics/kphotoalbum/AnnotationDialog/Dialog.cpp    (working copy)

<br>@@ -55,6 +55,7 @@<br> #include "ShowSelectionOnlyManager.h"<br> #include "enums.h"<br> #include "MainWindow/DirtyIndicator.h"<br>+#include <qtooltip.h><br> <br> AnnotationDialog::Dialog::Dialog( QWidget* parent, const char* name )

<br>     : QDialog( parent, name ), _viewer(0)<br>@@ -163,6 +164,7 @@<br>     lay6->addWidget( _copyPreviousBut );<br>     _copyPreviousBut->setIconSet( KGlobal::iconLoader()->loadIconSet( QString::fromLatin1( "legalmoves" ), KIcon::Desktop, 22 ) );

<br>     _copyPreviousBut->setFixedWidth( 40 );<br>+    QToolTip::add(_copyPreviousBut, i18n("Copy tags from last image"));<br>     connect( _copyPreviousBut, SIGNAL( clicked() ), this, SLOT( slotCopyPrevious() ) );

lay6->addStretch( 1 ); Apart from this, this feature seems ok.  I also had a look on the "show all when only one selected" and the "stop video playback" adds. No difference for me for the first (I have all my images on disk, so I guess I'm not the right person to test it :p), for the second one, just great ! I always thought it was a pity the sound didn't stop but never took the time to report it :-/.

<br><br>Thanks again, Jan.<br><br>-- <br>Baptiste <Batmat> MATHUS<br>BMathus at Batmat point net - <a href="http://batmat.net">http://batmat.net</a><br>---------<br>Si chacun de nous a une idée et que nous les partageons, nous

repartirons tous les deux avec deux idées... C'est ça le Libre.