[KPhotoAlbum] Delete select (images) dialog

[Baptiste MATHUS]

Hi all,

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 :-).

While working on it, I realized that the current delete dialog, even with
the patch removing the double checkbox isn't totally logical:
=> 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)?).

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.

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 :).

Thanks again.
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.

2007/5/19, Christoph Moseler <forums at moseler.net>:
>
> >>>      _block = new QCheckBox( i18n( "Block from database" ), top );
> >
> > If you change the label to "Don't delete, just block from database", you
> > have my ++ here :).
>
> "Block from database" was the original label, but I agree, your
> suggestion is more clear. Could you change the label before commiting my
> patch?
>
> Thanks,
> Christoph
> _______________________________________________
> KPhotoAlbum mailing list
> KPhotoAlbum at kdab.net
> http://mail.kdab.net/mailman/listinfo/kphotoalbum
>



-- 
Baptiste <Batmat> MATHUS
BMathus at Batmat point net - http://batmat.net
---------
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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kphotoalbum/attachments/20070519/c374cc3d/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: KPA-Delete+2.png
Type: image/png
Size: 13491 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kphotoalbum/attachments/20070519/c374cc3d/attachment.png>