unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Tomi Ollila <tomi.ollila@iki.fi>
To: David Bremner <david@tethera.net>,
	Mark Walters <markwalters1009@gmail.com>,
	notmuch@notmuchmail.org
Subject: Re: [PATCH] emacs: show: make id links respect window
Date: Sat, 05 Jan 2013 23:50:43 +0200	[thread overview]
Message-ID: <m2zk0ni9jg.fsf@guru.guru-group.fi> (raw)
In-Reply-To: <87bodgrh6a.fsf@zancas.localnet>

On Wed, Dec 26 2012, David Bremner <david@tethera.net> wrote:

> Mark Walters <markwalters1009@gmail.com> writes:
>
>>> I can imagine that people would want/like the "open in other window" 
>>> effect of the current code, even if the reason is a bug.
>>
>> That's definitely possible. I generally expect a mouse click to select
>> the window I click and this feels counter intuitive. I think that some
>> people might like an option "open this link in a new window" but I would
>> guess that would like that whether they clicked or pressed RET on the
>> button.
>
> I don't care much either way myself, but before we change notmuch-show
> behaviour (effectively) to accomodate notmuch-pick, I'd like a bit more
> feedback from other people.

I tested the old behaviour -- split frame to 2 windows, one containing
*scratch* and one *notmuch-hello*. Then I searched for 'obsoletes',
chose last message (id:1356936162-2589-1-git-send-email-amdragon@mit.edu)
moved point to *scratch* buffer and clicked the id: link -- and indeed,
the *scratch* window was replaced.

I didn't look or test Mark's patch as he stated:
"The lisp is not pretty but seems to work." ;)
... well, not entirely because of that but I trust it opens the message
in window where the clicked link were and keeps point where it used to
be before clicking (in case point was in different window).

So, personally I'd like to see this fixed but I'm even less qualified
to comment the implementation in the patch.


> d

Tomi

  reply	other threads:[~2013-01-05 21:50 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-19 23:10 [PATCH] emacs: show: make id links respect window Mark Walters
2012-12-24 15:54 ` David Bremner
2012-12-25 11:04   ` Mark Walters
2012-12-26 15:01     ` David Bremner
2013-01-05 21:50       ` Tomi Ollila [this message]
2013-01-06 15:43         ` Mark Walters
2013-01-06 15:56           ` Tomi Ollila
2013-01-06 16:10             ` Mark Walters
2013-01-07  7:04 ` Austin Clements

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=m2zk0ni9jg.fsf@guru.guru-group.fi \
    --to=tomi.ollila@iki.fi \
    --cc=david@tethera.net \
    --cc=markwalters1009@gmail.com \
    --cc=notmuch@notmuchmail.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).