From: Carl Worth <cworth@cworth.org>
To: Pablo Oliveira <pablo@sifflez.org>, notmuch@notmuchmail.org
Subject: Re: [PATCH] Goto-Address mode hides the body toggle button on messages of zero depth.
Date: Mon, 08 Feb 2010 14:54:43 -0800 [thread overview]
Message-ID: <87wrynxrrg.fsf@yoom.home.cworth.org> (raw)
In-Reply-To: <87aax0ocge.fsf@sifflez.org>
[-- Attachment #1: Type: text/plain, Size: 2198 bytes --]
On Wed, 30 Dec 2009 13:47:45 +0100, Pablo Oliveira <pablo@sifflez.org> wrote:
> Hello all,
Olá Pablo! Bem-vindo a notmuch!
> I heard about notmuch some weeks ago and I'm already liking it
> a lot. At first I was a little disapointed by the slow tagging,
> but that was fixed by Kan-Ru's great patch for Xapian. I have
> started using it daily, so many thanks for this mail reader !
I'm delighted to hear you're having a good experience with notmuch. And
I apologize that I wasn't able to reply any sooner. (Perhaps I can
excuse myself somewhat since the original message arrived on my wedding
anniversary? No I'm still *really* late.)
> One of the things I find counter-intuitive is that on closed messages
> of zero depth, the toggle body button is hidden by the email address
> button, so when working with the keyboard, it seems one cannot open
> the message, without moving further on the header button.
Yes, this is really annoying. I'd like to see a proper fix for this.
> I propose above fix because it is very simple, but I do not like that
> we lose one level of indentation on all the threads (screen space
> is precious). Another solution would be to change the name-email
> order in the header... Or maybe we should deactivate somehow, the
> Goto-Address mode on headers ? What do you think ?
For me, I'm getting the sender's name before their email address, so I
only run into this problem in the case where there is no name displayed,
but only an email address. Is that the same for you?
I don't want to remove the goto-address-mode on the header, because it
is very convenient (for those people who use a mouse) to be able to just
click on an email address to begin composing a new message to that
address.
I think I might be willing to have the email address deactivated for
keypresses if that were possible (so that pressing Enter anywhere on the
row would reliably toggle the message).
In the meantime, I agree that incrementing the depth universally doesn't
make sense. How about simply displaying a single ' ' character when
there's no full name available? I think that might solve the problem for
most people.
-Carl
[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]
prev parent reply other threads:[~2010-02-08 22:54 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-12-30 12:25 [PATCH] Goto-Address mode hides the body toggle button on messages of zero depth Pablo Oliveira
2009-12-30 12:47 ` Pablo Oliveira
2010-02-08 22:54 ` Carl Worth [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://notmuchmail.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87wrynxrrg.fsf@yoom.home.cworth.org \
--to=cworth@cworth.org \
--cc=notmuch@notmuchmail.org \
--cc=pablo@sifflez.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://yhetil.org/notmuch.git/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).