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 607549 - Next/Prev Message accelerators steal focus from preview pane
Next/Prev Message accelerators steal focus from preview pane
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.30.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
evolution[kill-bonobo]
Depends on:
Blocks:
 
 
Reported: 2010-01-20 13:15 UTC by Matthew Barnes
Modified: 2010-05-12 17:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Initial patch (809 bytes, patch)
2010-01-20 13:17 UTC, Matthew Barnes
rejected Details | Review

Description Matthew Barnes 2010-01-20 13:15:45 UTC
Originally posted to evolution-patches@gnome.org by Jan Holesovsky:

When using a mail client, I want to control it only using keyboard.  In
Evolution, the possibilities to move inside the message, and between
messages, include the Up and Down arrows (to move inside the message)
and Ctrl+], Ctrl+[, Ctrl+PgDn, and Ctrl+PgUp (to move between the
messages).

The problem is that any of Ctrl+], Ctrl+[, Ctrl+PgDn, Ctrl+PgUp steals
the focus from the message view, and gives it to message list - in which
case the meaning of Up and Down changes to moving between the messages
which I do not want - that is what Ctrl+PgDn and Ctrl+PgUp is for ;-) -
and you have to press Tab to get to the message view again.

The attached patch removes stealing of the focus - with that, you can
stay in the message preview & browse all your mail quickly and
effectively.

Please consider for inclusion.
Comment 1 Matthew Barnes 2010-01-20 13:17:00 UTC
Created attachment 151831 [details] [review]
Initial patch

Please forget this one, it is not good ;-) [focus stolen when you delete a 
message etc.] - I'll probably try another approach to achieve this, without 
such an offensive change.
Comment 2 Milan Crha 2010-05-12 17:42:03 UTC
Works fine with 2.30.1, it got fixed meanwhile, it seems. Please reopen, if you've issue with 2.30.x. Thanks in advance.