From: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
To: Radu Butoi <rbutoi@gmail.com>, notmuch@notmuchmail.org
Subject: Re: [PATCH] emacs: split-window-sensibly in tree mode with open message
Date: Thu, 21 May 2020 17:56:46 -0400 [thread overview]
Message-ID: <87367tj6jl.fsf@fifthhorseman.net> (raw)
In-Reply-To: <prg8o8r5dgiq.fsf@butoi.nyc.corp.google.com>
[-- Attachment #1.1: Type: text/plain, Size: 1184 bytes --]
On Sat 2020-05-02 20:11:09 -0400, Radu Butoi wrote:
> This uses the standard Emacs function `split-window-sensibly` to split a
> window horizontally or vertically depending on space when opening a
> message in tree view. By default, split-width-threshold is 160 columns
> (and -height- is nil), so screens wider than 160 will be split
> horizontally.
>
> This is based on an older proposal [1] which manually did the
> calculation of width. The main issues there were (1) lack of
> configurability and (2) lack of testing. I don't have an answer for
> testing, but this allows users to configure two thresholds using
> built-in variables, an improvement.
I like this proposal, and the simplification that it gives to the
notmuch-emacs codebase. However, this thread is the first place i've
learned about split-window-sensibly, so i'm probably not eligible to
really judge the merits here.
As far as testing goes, a test would be nice -- is this something you
could add to test/T460-emacs-tree.sh ? Testing UI/UX issues is always
pretty tough though, and it's not clear to me that we're actually
already testing the existing "(/ (window-height) 4)" business anyway.
--dkg
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
next prev parent reply other threads:[~2020-05-21 22:12 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-03 0:11 [PATCH] emacs: split-window-sensibly in tree mode with open message Radu Butoi
2020-05-03 11:19 ` David Bremner
2020-05-09 19:14 ` Radu Butoi
2020-05-21 21:56 ` Daniel Kahn Gillmor [this message]
2020-05-22 5:03 ` Radu Butoi
2020-05-22 18:58 ` Mark Walters
2020-05-23 19:32 ` Daniel Kahn Gillmor
2020-05-26 18:48 ` Radu Butoi
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=87367tj6jl.fsf@fifthhorseman.net \
--to=dkg@fifthhorseman.net \
--cc=notmuch@notmuchmail.org \
--cc=rbutoi@gmail.com \
/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).