[Evolution-hackers] [UI] attachment dropdown focus hijacking (mouse)

[guenther <guenther rudersport de>]

cheers hacker,

Just encountered this UI bug and filed it:
 http://bugzilla.ximian.com/show_bug.cgi?id=49646

This is a serious regression and makes handling of attachments a PITA.
What do you think about this?

...guenther


--- snip ---

Clicking the attachmant dropdown in a mail (for example to display the
attached message) hijacks the focus. That means, the focus which was
previously at the mail body to use scrolling by key, isn't any longer at
the mail body.

This is a *regression* since 1.4.5, last version without this bug is
1.4.4 (official versions, don't know about snapshots).

As this is a regression and a serious UI bug, I consider this major
priority.


Steps to reproduce the problem:

- have a mail with an attachment (mail here)
- click the mail body (preview pane)
  - focus is at the mail body
  - scrolling works, using the up/down keys
- click the attachment dropdown (mouse), attachments gets displayed
  - focus is hijacked by the dropdown
  - up/down keys do *not* what the user expects [1]


Expected Results:

- return the focus back to the mail body


Additional Info:

During the discussion in IRC, these bugs where noted -- maybe the
patches have introduced this bug:
 #41839  Keyboard: Can't focus on attachment in mail body.
 #47224  dragging of attachments broken


[1]  the down key cycles focus:  -> search input field -> mail list
(selected message) -> mail list, selects messages below


-- 
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; }}}