Hi all,<br><br>I've been busy today. I'm now back and I've tried to integrate the comments from Jan. Sorry, but I still need your attention and opinions :-).<br><br>While working on it, I realized that the current delete dialog, even with the patch removing the double checkbox isn't totally logical: 
<br>=> the checkbox would contain "Don't delete, only block from database". so you could check it and click on... "Delete". I think that's not the most simple way to be understood :-), mainly for new users (I fear new users are going to wonder: "wait a minute, if I click on delete while having checked the upper box, will it anyway delete permanently my image(s)?). 
<br><br>So, I thought about using "Proceed" text instead in the button. It seems a bit clearer, but maybe it's not the best way to go. I also lowered the font size from +3 to +2. Imo, it looks a bit better and still an important message.
<br><br>A screenshot of the resulting still uncommitted delete dialog is attached. Please advice. If I got two Yay about the dialog in the attachment, then I will commit it very soon :).<br><br>Thanks again.<br>PS: if you prefer, I could have committed it so as to simplify real world reviews. I guess that (1) Jesper and Tuomas could give his opinion about and (2) this is the sort of thing we can afford as it is the trunk.
<br><br><div><span class="gmail_quote">2007/5/19, Christoph Moseler <<a href="mailto:forums@moseler.net">forums@moseler.net</a>>:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
>>>      _block = new QCheckBox( i18n( "Block from database" ), top );<br>><br>> If you change the label to "Don't delete, just block from database", you<br>> have my ++ here :).<br>
<br>"Block from database" was the original label, but I agree, your<br>suggestion is more clear. Could you change the label before commiting my<br>patch?<br><br>Thanks,<br>Christoph<br>_______________________________________________
<br>KPhotoAlbum mailing list<br><a href="mailto:KPhotoAlbum@kdab.net">KPhotoAlbum@kdab.net</a><br><a href="http://mail.kdab.net/mailman/listinfo/kphotoalbum">http://mail.kdab.net/mailman/listinfo/kphotoalbum</a><br></blockquote>
</div><br><br clear="all"><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
<br>repartirons tous les deux avec deux idées... C'est ça le Libre.