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 127267 - [PATCH] Eel stock dialogs do not follow HIG
[PATCH] Eel stock dialogs do not follow HIG
Status: VERIFIED FIXED
Product: eel
Classification: Deprecated
Component: general
unspecified
Other Linux
: High normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-11-18 07:25 UTC by Dennis Cranston
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed Patch. (19.71 KB, patch)
2003-11-18 07:26 UTC, Dennis Cranston
none Details | Review
New proposed header file. (2.33 KB, text/plain)
2003-11-18 07:27 UTC, Dennis Cranston
  Details
New proposed source file. (13.04 KB, text/plain)
2003-11-18 07:27 UTC, Dennis Cranston
  Details
[Part One] Proposed eel patch (updated). (19.95 KB, patch)
2003-11-26 22:01 UTC, Dennis Cranston
none Details | Review
Proposed new eel header file (updated). (2.33 KB, patch)
2003-11-26 22:02 UTC, Dennis Cranston
none Details | Review
Proposed new eel source file (updated). (13.04 KB, patch)
2003-11-26 22:02 UTC, Dennis Cranston
none Details | Review
[Part Two] Proposed nautilus patch. (109.54 KB, patch)
2003-11-26 22:03 UTC, Dennis Cranston
none Details | Review
A sample screenshot of one dialog. (22.05 KB, image/png)
2003-11-26 22:10 UTC, Dennis Cranston
  Details

Description Dennis Cranston 2003-11-18 07:25:37 UTC
The dialogs provided by eel-stock-dialogs.c are not HIG compliant.  

See
http://developer.gnome.org/projects/gup/hig/1.0/windows.html#alert-windows
for details on HIG compliant alerts.
Comment 1 Dennis Cranston 2003-11-18 07:26:25 UTC
Created attachment 21569 [details] [review]
Proposed Patch.
Comment 2 Dennis Cranston 2003-11-18 07:27:09 UTC
Created attachment 21570 [details]
New proposed header file.
Comment 3 Dennis Cranston 2003-11-18 07:27:51 UTC
Created attachment 21571 [details]
New proposed source file.
Comment 4 Dennis Cranston 2003-11-18 07:30:05 UTC
The attached files implement the eel half of this proposed change.  I
will attach a patch for nautilus half later in the week. 
Comment 5 Dennis Cranston 2003-11-26 22:01:19 UTC
Created attachment 21842 [details] [review]
[Part One]  Proposed eel patch (updated).
Comment 6 Dennis Cranston 2003-11-26 22:02:16 UTC
Created attachment 21843 [details] [review]
Proposed new eel header file (updated).
Comment 7 Dennis Cranston 2003-11-26 22:02:45 UTC
Created attachment 21844 [details] [review]
Proposed new eel source file (updated).
Comment 8 Dennis Cranston 2003-11-26 22:03:41 UTC
Created attachment 21845 [details] [review]
[Part Two]  Proposed nautilus patch.
Comment 9 Dennis Cranston 2003-11-26 22:07:49 UTC
The attached patch files modify eel and nautilus to display HIG
compliant alert dialogs.  I modified the eel stock dialogs to support
setting primary and secondary message text.  Feedback is much appreciated.

Thanks,
Dennis
Comment 10 Dennis Cranston 2003-11-26 22:10:23 UTC
Created attachment 21846 [details]
A sample screenshot of one dialog.
Comment 11 Mark Finlay 2003-11-28 19:09:21 UTC
Marking priority high because there is a patch, and ccing usability
incase anyone wants to check this.

Dennis, you rock!
Comment 12 John McCutchan 2003-11-28 23:39:36 UTC
There is a major bug with this new dialog. You can't have a dialog
asking you to overwrite a file that doesn't properly display the full
filename of the target file. Even better would be something that tells
you the size/name/date of the source file and the size/name/date of
the target file. The new (and old) dialog is useless for an average user.
Comment 13 Dennis Cranston 2003-11-29 01:27:47 UTC
I agree. The overwrite dialog is pretty useless, but the point of the
proposed patch is to make the various alert dialogs in eel HIG
compliant.  

I feel what you are requesting may be beyond the scope of this bug
report.  Maybe a new bug report should be opened for the overwrite
dialog?  (If one has not been opened already.)  If I recall correctly,
Mark has a web page with some great suggestion for the overwrite file
dialog. Unfortunately, I can't seem to find the link. :-(  
Comment 14 Dennis Cranston 2003-11-29 01:45:24 UTC
BTW, to clarify.  The ellipsized file name is nothing new.  The old
dialog does the same thing.

The ellipsizing could be disabled by changing the following code in
nautilus-file-operations.c.

@@ -1039,39 +1052,42 @@ handle_transfer_overwrite (const GnomeVF
 	}
 	
 	/* transfer conflict, prompt the user to replace or skip */
-	formatted_name = format_and_ellipsize_uri_for_dialog (
-		parent_for_error_dialog (transfer_info), progress_info->target_name);
+	formatted_name = eel_format_uri_for_display
(progress_info->target_name);
Comment 15 Mark Finlay 2003-11-29 09:24:12 UTC
I agree with Dennis. These dialogs need a _lot_ of usability work[1]
but that is outside the scope of this bug. That will be a lot easier
to do once we have these HIG patches in.

What we need is a to all go through these dialogs looking for
usability problems, and then file bugs(cc me please). Another thing
that would be REALLY useful would be a screenshot of every possible
eror/alert dialog with a description of what was done to generate
them. With that I/We could design some really nice dialogs.

[1] eg. If you cancel a move the files you tried to move are copied
anyway. More details are on an old blog of mine at
http://sisob.tuxfamily.org/archives/000078.html - maybe that's the one
Dennis was thinking of.
Comment 16 Dennis Cranston 2003-11-29 18:14:30 UTC
Yes, that is the web page.  
Thanks for reposting the link.
Comment 17 Alexander Larsson 2003-12-11 10:37:05 UTC
Ok, i commited something based on this.
I made some changes to the implementation of eel-alert-dialog, and
made it use GtkExpander for the details. I also fixed some bugs in the
nautilus patch, changed which strings were primary/secondary in some
places and,  changed a few messages.

Comment 18 Dennis Cranston 2003-12-11 15:13:04 UTC
Great!  Thanks Alex.