From 083045ad530f678ea95558a6a12ca4823c576f25 Mon Sep 17 00:00:00 2001 From: Junegunn Choi Date: Sat, 18 Jul 2026 00:52:44 +0900 Subject: [PATCH] Clarify comment on height overestimation --- src/options.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/options.go b/src/options.go index 3e92c6b9..f67583c9 100644 --- a/src/options.go +++ b/src/options.go @@ -3696,9 +3696,9 @@ func (opts *Options) noSeparatorLine() bool { if opts.Inputless { return true } - // NOTE: This can overestimate by one line when the default separator is - // suppressed at runtime (see Terminal.separatorLength), making the height - // lower bounds derived from it one line more generous than necessary. + // NOTE: This does not know that the default separator can be suppressed at + // runtime (see Terminal.separatorLength), so the minimum heights derived + // from it can be one line taller than strictly necessary. sep := opts.Separator == nil && !opts.InputBorderShape.Visible() || opts.Separator != nil && len(*opts.Separator) > 0 return noSeparatorLine(opts.InfoStyle, sep) }