Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Oskari Pirhonen
On Mon, Oct 18, 2021 at 12:33:33AM +0200, Matthias Andree wrote: > 1. Is the "clear only if mutt exits with status 0" intentional? If not, > ";" would be preferable to "&&". It is intentional, yes. I don't want it to clear the screen if there's an error so that any messages that may be printed are

Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Matthias Andree
Am 17.10.21 um 22:04 schrieb Oskari Pirhonen: > On Sun, Oct 17, 2021 at 08:31:41PM +0100, Claus Assmann wrote: >> Why don't you use a wrapper which invokes >> clear >> after mutt finished? No extra code needed in mutt. > Wow, I can't believe I didn't think of that *facepalms*. > > Thanks for the ti

Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Oskari Pirhonen
On Sun, Oct 17, 2021 at 08:31:41PM +0100, Claus Assmann wrote: > Why don't you use a wrapper which invokes > clear > after mutt finished? No extra code needed in mutt. Wow, I can't believe I didn't think of that *facepalms*. Thanks for the tip. It was much faster to write than adding the config o

Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Claus Assmann
Why don't you use a wrapper which invokes clear after mutt finished? No extra code needed in mutt.

Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Oskari Pirhonen
On Sun, Oct 17, 2021 at 09:35:26AM -0700, Kevin J. McCarthy wrote: > I think most terminals nowadays have alternative screen support enabled > by default, don't they? I'm not sure what you mean by that? All I know is that when I run mutt in a TTY (my main interface) it doesn't clear the screen, b

Re: [PATCH] Option to clear the screen on quit

2021-10-17 Thread Kevin J. McCarthy
On Sun, Oct 17, 2021 at 12:25:24AM -0500, Oskari Pirhonen wrote: Add the option to clear the screen when quitting mutt. This is controlled by the clear_on_quit config option. Defaults to "no". I think most terminals nowadays have alternative screen support enabled by default, don't they? Unle