unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: David Bremner <david@tethera.net>
To: Aaron Ecay <aaronecay@gmail.com>,
	Jameson Graef Rollins <jrollins@finestructure.net>,
	Thomas Jost <schnouki@schnouki.net>,
	notmuch@notmuchmail.org
Subject: Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails
Date: Mon, 26 Dec 2011 07:38:52 -0400	[thread overview]
Message-ID: <87boqvy33n.fsf@zancas.localnet> (raw)
In-Reply-To: <m2ty4oszji.fsf@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2054 bytes --]

On Sun, 25 Dec 2011 23:54:41 -0500, Aaron Ecay <aaronecay@gmail.com> wrote:

> I just tried, and I cannot reproduce this behavior.  IIUC, here is what
> happened to you: you set nm-mua-compose-in to 'new-window.  You began a
> new message, this opened a new window as expected.  Your emacs frame now
> has two windows in it.  You sent this message, which deleted the window
> showing it.  Your emacs frame was deleted as well, which made the other
> window, showing notmuch-hello (or some other notmuch buffer, from which
> you began writing the email message) disappear as well, unexpectedly.
> Is this a correct description of what happened?

Yes, although it happened quickly enough I'm not sure the window was
deleted before the frame.

> Here’s the recipe I used for replicating:
> 
> emacs -q --daemon
> emacsclient -c
> C-x b *scratch*
> (add-to-list 'load-path "/path/to/notmuch/emacs/") C-j
> (load-library "notmuch") C-j
> C-x C-f /path/to/notmuch/emacs/notmuch-mua.el
> M-x eval-buffer (in order to pick up changes not in byte-compiled file)
> M-x customize-variable notmuch-mua-compose-in (set to 'new-window, save for session)
> M-x notmuch
> m (new window is created in current frame, below the window showing notmuch-hello)
> (type mail)
> C-c C-c (enter smtp settings, since emacs doesn’t know them)
> (new window disappears, the window with notmuch-hello fills whole frame)
> 

It sounds plausible. On Debian I was a bit lazy and relied on the Debian
site startup file, which I attach.  Note that the setting of 'new-window
seems broken to me without emacsclient as well. In this case it buries
the notmuch-hello buffer that the compose window was launched from.

> I also tried with notmuch-mua-compose-in set to 'new-frame, and got the
> expected behavior (m -> create new frame, C-c C-c -> new frame is
> deleted)

Yes, that one seems fine; perhaps because deleting the frame is the
desired outcome in this case.

> What version of emacs did you have this problem with?  

23.3.1



[-- Attachment #2: site startup for notmuch. --]
[-- Type: application/emacs-lisp, Size: 1302 bytes --]

  reply	other threads:[~2011-12-26 11:39 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-03 12:03 [PATCH 0/4] Several minor Emacs enhancements Thomas Jost
2011-10-03 12:03 ` [PATCH 1/4] emacs: Add a face for crypto parts headers Thomas Jost
2011-10-03 12:03 ` [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua Thomas Jost
2011-10-10 15:50   ` Jameson Graef Rollins
2011-10-10 18:25     ` Jameson Graef Rollins
2011-10-11 19:49     ` Thomas Jost
2011-10-11 19:51       ` [PATCH v2] " Thomas Jost
2011-10-18 18:58         ` Jani Nikula
2011-10-18 14:46       ` [PATCH 2/4] " Jameson Graef Rollins
2011-10-25  7:39         ` Thomas Jost
2011-10-25  7:41           ` [PATCH] emacs: Let the user choose where to compose new mails Thomas Jost
2011-11-05  3:51             ` Austin Clements
2011-11-09 18:50               ` Thomas Jost
2011-11-09 20:06                 ` [PATCH v2] " Thomas Jost
2011-11-13 21:41                   ` Jameson Graef Rollins
2011-10-03 12:03 ` [PATCH 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator Thomas Jost
2011-10-03 12:03 ` [PATCH 4/4] emacs: add notmuch-hello-hook Thomas Jost
2011-11-12 16:39 ` [PATCH 0/4] Several minor Emacs enhancements David Bremner
2011-12-13 17:32   ` [PATCH v3 " Thomas Jost
2011-12-13 17:32     ` [PATCH v3 1/4] emacs: Let the user choose where to compose new mails Thomas Jost
2011-12-15 11:27       ` David Bremner
2011-12-15 17:18         ` Jameson Graef Rollins
2011-12-15 23:50           ` David Bremner
2011-12-16 23:45             ` Jameson Graef Rollins
2011-12-17  1:19               ` David Bremner
2011-12-17  1:35                 ` Jameson Graef Rollins
2011-12-18 18:46                 ` Tom Prince
2011-12-26  4:54             ` Aaron Ecay
2011-12-26 11:38               ` David Bremner [this message]
2012-01-06 16:45               ` Thomas Jost
2012-04-14 19:36       ` Jameson Graef Rollins
2012-04-14 20:20         ` [PATCH] " Jameson Graef Rollins
2012-04-14 20:42           ` [PATCH v5] " Jameson Graef Rollins
2012-04-28  7:23           ` [PATCH] " Tomi Ollila
2012-04-15 14:52         ` [PATCH v3 1/4] " David Bremner
2012-04-22 22:25           ` Thomas Jost
2012-04-29 19:12             ` David Bremner
2012-05-04 10:37               ` [PATCH v6] " Thomas Jost
2012-05-05 23:20                 ` Jameson Graef Rollins
2012-05-06 12:21                 ` David Bremner
2011-12-13 17:32     ` [PATCH v3 2/4] emacs: Add a face for crypto parts headers Thomas Jost
2011-12-16  0:56       ` Dmitry Kurochkin
2011-12-16  3:04       ` David Bremner
2011-12-13 17:32     ` [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator Thomas Jost
2011-12-16  0:59       ` Dmitry Kurochkin
2011-12-16 12:29         ` David Bremner
2011-12-16 12:34           ` Dmitry Kurochkin
2011-12-21  0:30             ` Thomas Jost
2011-12-21  1:41               ` Dmitry Kurochkin
2011-12-21 13:44                 ` [PATCH 1/2] emacs: rename notmuch-decimal-separator to notmuch-hello-thousands-separator Thomas Jost
2011-12-21 13:44                   ` [PATCH 2/2] emacs: Change the default thousands separator to a space Thomas Jost
2011-12-21 14:12                     ` Tomi Ollila
2011-12-21 16:37                   ` [PATCH 1/2] emacs: rename notmuch-decimal-separator to notmuch-hello-thousands-separator Dmitry Kurochkin
2011-12-22 11:25                   ` David Bremner
2011-12-13 17:32     ` [PATCH v3 4/4] emacs: add notmuch-hello-hook Thomas Jost
2011-12-13 17:49       ` Dmitry Kurochkin
2011-12-16 18:36         ` Dmitry Kurochkin
2011-12-21  0:21           ` [PATCH] emacs: add notmuch-hello-refresh-hook Thomas Jost
2011-12-21  1:28             ` Thomas Jost
2011-12-21 12:03               ` David Bremner
2011-12-21 12:17               ` David Bremner
2011-12-21  1:33             ` Dmitry Kurochkin
2011-12-21 13:13               ` [PATCH] test: add tests for `notmuch-hello-refresh-hook' Thomas Jost
2011-12-21 18:26                 ` Dmitry Kurochkin
2011-12-13 18:16     ` [PATCH v3 0/4] Several minor Emacs enhancements Jameson Graef Rollins

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=87boqvy33n.fsf@zancas.localnet \
    --to=david@tethera.net \
    --cc=aaronecay@gmail.com \
    --cc=jrollins@finestructure.net \
    --cc=notmuch@notmuchmail.org \
    --cc=schnouki@schnouki.net \
    /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).