Mime-Version: 1.0
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain; charset=UTF-8
From: "Robin Jarry" <robin@jarry.cc>
To: "James Cook" <falsifian@falsifian.org>, <~rjarry/aerc-devel@lists.sr.ht>
Subject: Re: [PATCH aerc] Allow not marking viewed messages as seen.
Date: Sun, 10 Jul 2022 20:25:52 +0200
Message-Id: <CLC6R7U7HXHI.HWEUHR6XAYV5@marty>
User-Agent: aerc/0.10.0-62-g0e889045e8b8-dirty
References: <20220705060618.68685-1-falsifian@falsifian.org>
In-Reply-To: <20220705060618.68685-1-falsifian@falsifian.org>
Cc: 

James Cook, Jul 05, 2022 at 08:06:
> Fixes: https://todo.sr.ht/~rjarry/aerc/48
> Signed-off-by: James Cook <falsifian@falsifian.org>
> ---
> Sending this out for feedback. I should probably test it at least a
> little more; e.g. I haven't tried deleting a message to check that the
> change to delete.go makes sense.

Hey James,

The concept looks nice.

Having the option in UI is weird but it allows contextual configuration
(per-account, folder, etc.), so I guess I can live with it. I was never
fond of this dependency between aerc.conf and accounts.conf. Ideally
these two files should be completely independent. Any account/folder
specific stuff should be in accounts.conf.

Anyways, I agree with Tim about conditionally setting the Peek flag when
fetching the message body in IMAP when auto-mark-read=3Dfalse. That way,
no need to worry about setting or un-setting that \Seen flag.

Cheers