Re: [Evolution-hackers] Search in message/Find in message



On Thu, 2005-06-23 at 11:05 -0400, Lee Revell wrote:
> I'm using a CVS build of Evo 2.3.1, and noticed that the mail reader
> uses Ctrl-S/Search in Message, while the composer uses Ctrl-F/Find in
> message.
> 
> I still think using Ctrl-S is a big usability problem, it has taken me
> weeks to get used to, as Firefox and IE use Ctrl-F, but it's definitely
> not good to have separate bindings for the reader and composer.

Well, MUAs, browsers, Office apps, etc. are totally different apps, with
totally different needs for proper accelerators.

Ctrl-F in the Mail Component is Forwarding a message. It has been like
that for a long time (always?), and it definitely should not change
(read: break) for the users.

At least my (personal) use case for "searching" does not involve "Search
in Message". In fact, IIRC I used it about 2 times during all those
years I'm very actively using and supporting Evolution. The average Mail
isn't that large, that I cannot easily spot what I'm looking for inside
the mail. The search action I *do* use often though is searching for a
particular mail that contains special words inside one (or more) large
folders full of mails... The search bar. :-)


Anyway, back to your issue: If both of those "Search" or "Find" actions
should be more consistent, it really should be "Search".

* We definitely should not change the Forward mail accelerator in the
  Mail Component.
* Granted, Ctrl-S currently is used in the Composer. But that's not a
  real concern to me. Saving mails out of the Composer is a broken
  option anyway, as there is no way to easily "Load" that saved mail...
  (as opposed to save as a draft)
  IMHO the File / Save option in the Composer can be removed. [1]

...guenther


[1] http://bugzilla.gnome.org/show_bug.cgi?id=309019


-- 
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]