[KPhotoAlbum] Delete select (images) dialog

[Tuomas Suutari]

On Saturday 19 May 2007 19:08, Baptiste MATHUS wrote:
> Hi all,

Hi!


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

I would put it other way around, so the dialog would look like this

+-------------------------------------------+
| Remove files from the database            |
+-------------------------------------------+
|                                           |
| You are about to remove %n files from the |
| database. Do you really want to do it?    |
|                                           |
| [ ] Also move the files to trash.         |
|                                           |
+-------------------------------------------+

> 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

-- 
Tuomas Suutari | +358 50 3806983 | thsuut at utu.fi