[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mutt-dev
Subject:    Re: [PATCH] Add skip_quoted_context option
From:       "Kevin J. McCarthy" <kevin () 8t8 ! us>
Date:       2021-06-17 14:12:37
Message-ID: YMtYVUaCcfXRzCJd () afu ! lan
[Download RAW message or body]


On Thu, Jun 17, 2021 at 10:34:47AM +0200, Rene Kita wrote:
>On Mon, Jun 14, 2021 at 07:44:24PM -0700, Kevin J. McCarthy wrote:
>> I'm attaching a modified version of your patch that:
>>
>> - renames the variable to $pager_skip_quoted_context.
>The commit message needs to be updated to reflect this rename.

Whoops I missed that.  :-)  Good catch!

>Now using <skip-quoted> and
>- set pager_skip_quoted_context=4 -> shows 2 lines, both wrapped.
>- set pager_skip_quoted_context=2 -> shows 1 line which is wrapped
>- set pager_skip_quoted_context=1 -> shows the wrapped part only

I think we're saying the same thing.  I understood the original line in 
the commit message, "Wrapped lines do not count towards 
skip_quoted_context.", to mean the wrapped line doesn't count towards 
the total display lines shown as context.  Meaning in the first case, 
all three quoted lines would be displayed (for a total of 6 display 
lines), etc.

Thanks for testing.  I will fix the commit message and push it up later 
today!

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]