Re: [evolution-patches] Proposed UI change: mailer component: ellipsis characters for various menu items (#323951)



On Tue, 2005-12-13 at 01:47 -0500, David Malcolm wrote:
> (trying to get some UI fixes in before the deadline here)
> 
> >From the GNOME HIG:
> "Command items are menu items that initiate a command or perform an action, such
> as Save, Print or Quit. They may act on the currently active document in a
> document based application, or on the application itself." ... "Label the menu
> item with a trailing ellipsis ("...") only if the command requires further input
> from the user before it can be performed. Do not add an ellipsis to items that
> only present a confirmation dialog (such as Delete), or that do not require
> further input (such as Properties, Preferences or About)." (see
> http://developer.gnome.org/projects/gup/hig/2.0/menus-design.html#menu-grouping 
> 
> 
> Based on the above, I think the following items in the mailer should have
> trailing ellipsis characters:
> Message ->
> 	Compose New Message
> 	Post New Message to Folder
> 	Reply
> 	Post a Reply
> 	Reply to Sender
> 	Reply to List
> 	Forward
> 	Redirect

Honestly, I disagree for the above. I don't think they should have the
ellipsis chars. (Yes, this mostly is a gut feeling of mine, seems crappy
to me to add it to all of em.) So I'd like to get this reverted.

Anyway, in either case... Be consistent!

* If the menu entry "Reply" has the ellipsis, the context menu needs it
  as well.
* If the above should have the ellipsis, the same applies to all menu
  entries in the File / New menu and New dropdown.
* Context menu for Calendar, Tasks, ... The list goes on and on.


IMHO it is way worse to use different styles all over the place, or to
add hundreds of dots to the UI, than just leaving this as is (was)...


> 	Move to Folder
> 	Copy to Folder
> 
> Attached is a patch that adds the ellipsis characters to these menu items.
> OK to commit? (with a ChangeLog entry)

...guenther


-- 
char *t="\10pse\0r\0dtu\0  ghno\x4e\xc8\x79\xf4\xab\x51\x8a\x10\xf4\xf4\xc4";
main(){ char h,m=h=*t++,*x=t+2*h,c,i,l=*x,s=0; for (i=0;i<l;i++){ i%8? c<<=1:
(c=*++x); c&128 && (s+=h); if (!(h>>=1)||!t[s+h]){ putchar(t[s]);h=m;s=0; }}}




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]