After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 309046 - delete confirmation dialog should be removed
delete confirmation dialog should be removed
Status: RESOLVED FIXED
Product: eog
Classification: Core
Component: collection
2.11.x
Other Linux
: Normal minor
: ---
Assigned To: EOG Maintainers
EOG Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-06-26 10:21 UTC by Teppo Turtiainen
Modified: 2005-11-09 03:38 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
screenshot of confirmation dialog (30.29 KB, image/png)
2005-06-26 10:23 UTC, Teppo Turtiainen
  Details
Implements Nautilus like dialog. (977 bytes, patch)
2005-06-26 15:17 UTC, Muthiah Annamalai
none Details | Review
fixed image. (97.26 KB, image/png)
2005-06-26 15:19 UTC, Muthiah Annamalai
  Details

Description Teppo Turtiainen 2005-06-26 10:21:51 UTC
Distribution/Version: Ubuntu Hoary

1. Open a bunch of images in the collection view.
2. Delete one of the images.
3. Look at the confirmaton dialog.

There is no title, it uses the warning icon instead of the question icon, the
buttons are in different order than in nautilus delete confirmation dialog, it
doesn't display the name of the file and the entire text is on one line.

Please see the attached screenshot. I included the nautilus confirmation dialog
for comparison.
Comment 1 Teppo Turtiainen 2005-06-26 10:23:09 UTC
Created attachment 48341 [details]
screenshot of confirmation dialog
Comment 2 Muthiah Annamalai 2005-06-26 14:32:31 UTC
Agreed. I think if it works like nautilus it'd be cool.
Making a patch on the latest CVS head.
Comment 3 Luis Villa 2005-06-26 14:40:45 UTC
I think Nautilus uses something from the eel library for this; maybe you could
talk with the nautilus maintainers about using eel for this? They tend to frown
on other people linking against eel, but if it is there, standardizing seems
like it would be useful.
Comment 4 Muthiah Annamalai 2005-06-26 15:15:14 UTC
Well, heres the patch and a file list.
I think it'd help if user sees a list of files
s/he is going to delete, not just 'move 2 files to trash',
say 'move $1, $2 to trash ?'
That'd be good, probl'y a separate bug?.
Comment 5 Muthiah Annamalai 2005-06-26 15:16:23 UTC
Well, heres the patch and a file list.
I think it'd help if user sees a list of files
s/he is going to delete, not just 'move 2 files to trash',
say 'move $1, $2 to trash ?'
That'd be good, probl'y a separate bug?.
Comment 6 Muthiah Annamalai 2005-06-26 15:17:44 UTC
Created attachment 48357 [details] [review]
Implements Nautilus like dialog.

Against latest CVS.
Comment 7 Muthiah Annamalai 2005-06-26 15:19:03 UTC
Created attachment 48358 [details]
fixed image.

A screen shot: How it looked on my systems
Comment 8 Teppo Turtiainen 2005-06-26 15:25:45 UTC
Looks good, thanks. About the file list: Nautilus doesn't show a list of files
either. What happens when the user tries to delete great many files at once? I
think it would be better for consistency to just mimic what Nautilus does.
Comment 9 Paolo Borelli 2005-06-26 18:49:50 UTC
A couple of comments:

- the dialog should not have a title according to the HIG (it was one of the few
things the old dialog got right)
- if someone wants to implement a dialog with a list,
gedit-close-confirmation-dialog.c contains code that can be used

a more generic question is: do we really need this dialog? (nautilus doesn't ask
for confirmation when moving to the trash)
Comment 10 Teppo Turtiainen 2005-07-08 04:37:27 UTC
I'm OK with removing the dialog entirely.
Comment 11 Tim Gerla 2005-07-09 18:35:47 UTC
I'm tending to agree with Paolo and Teppo; matching nautilus would be nice. I
think we can remove the dialog entirely.
Comment 12 Teppo Turtiainen 2005-08-10 06:21:26 UTC
Changing the summary and upping versions.
Comment 13 Lucas Rocha 2005-11-09 03:08:11 UTC
I agree about matching nautilus and removing the delete confirmation dialog.
Comment 14 Lucas Rocha 2005-11-09 03:38:34 UTC
Applied in HEAD. Delete (move to trash) confirmation dialog removed. Thanks!