fix: compensate for printing performance on Windows

Updating the screen takes a much longer time on Windows than it does on
other platforms. This change adds a longer update delay on Windows to
help to compensate for that.

Signed-off-by: Donnie Adams <[email protected]>

Author: thedadams

display.go

@@ -1,17 +1,19 @@
 package tui
 
 import (
+	"runtime"
 	"strings"
 	"time"
 
 	"github.com/pterm/pterm"
 )
 
 type display struct {
-	area     *pterm.AreaPrinter
-	prompter *prompter
-	last     time.Time
-	stopped  bool
+	area         *pterm.AreaPrinter
+	prompter     *prompter
+	last         time.Time
+	lastDuration int64
+	stopped      bool
 }
 
 func newDisplay(tool string) (*display, error) {
@@ -87,17 +89,21 @@ func (a *display) Progress(text string) error {
 		a.area = area
 		a.stopped = false
 		a.last = time.Time{}
+		a.lastDuration = 0
 	}
 
 	now := time.Now()
-	if now.Sub(a.last).Milliseconds() > 50 {
-		a.last = now
+	if now.Sub(a.last).Milliseconds() > a.lastDuration+50 {
 		lines := strings.Split(text, "\n")
 		height := pterm.GetTerminalHeight()
 		if len(lines) > height {
 			lines = lines[len(lines)-height:]
 		}
 		a.area.Update(strings.Join(lines, "\n"))
+		if !a.last.IsZero() && runtime.GOOS == "windows" {
+			a.lastDuration = time.Now().Sub(a.last).Milliseconds()
+		}
+		a.last = now
 	}
 
 	return nil

run.go

@@ -102,9 +102,11 @@ func Run(ctx context.Context, tool string, opts ...RunOptions) error {
 		var text string
 
 		for event := range run.Events() {
-			text = render(input, run)
-			if err := ui.Progress(text); err != nil {
-				return err
+			if event.Call != nil {
+				text = render(input, run)
+				if err := ui.Progress(text); err != nil {
+					return err
+				}
 			}
 
 			if ok, err := confirm.HandlePrompt(ctx, event, ui.Ask); !ok || err != nil {