From 8ce6240c02640023580e31a6a2c91c972dd5ea97 Mon Sep 17 00:00:00 2001
From: Stu Leak <leaktechnologies@proton.me>
Date: Sun, 28 Dec 2025 22:21:09 -0500
Subject: [PATCH] Hide benchmark indicator in Convert module when already
 applied
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

User feedback: Don't show benchmark status clutter when settings are already applied.

Changes:
- Only show benchmark indicator when settings are NOT applied
- Removes 'Benchmark: Applied' text + button from UI when active
- Cleaner Convert module interface when using benchmark settings

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---
 main.go | 37 ++++++++++++++++---------------------
 1 file changed, 16 insertions(+), 21 deletions(-)

diff --git a/main.go b/main.go
index 0cd8660..fdff53c 100644
--- a/main.go
+++ b/main.go
@@ -6297,30 +6297,25 @@ func buildConvertView(state *appState, src *videoSource) fyne.CanvasObject {
 			state.convert.EncoderPreset == preset &&
 			state.convert.HardwareAccel == benchHW
 
-		statusColor := utils.MustHex("#FFC857")
-		statusText := "Benchmark: Not Applied"
-		if applied {
-			statusColor = utils.MustHex("#4CE870")
-			statusText = "Benchmark: Applied"
-		}
-		benchmarkStatus = canvas.NewText(statusText, statusColor)
-		benchmarkStatus.TextStyle = fyne.TextStyle{Monospace: true, Bold: true}
-		benchmarkStatus.TextSize = 12
+		// Only show benchmark indicator if settings are NOT already applied
+		if !applied {
+			statusColor := utils.MustHex("#FFC857")
+			statusText := "Benchmark: Not Applied"
+			benchmarkStatus = canvas.NewText(statusText, statusColor)
+			benchmarkStatus.TextStyle = fyne.TextStyle{Monospace: true, Bold: true}
+			benchmarkStatus.TextSize = 12
 
-		benchmarkApplyBtn = widget.NewButton("Apply Benchmark", func() {
-			state.applyBenchmarkRecommendation(encoder, preset)
-			benchmarkStatus.Text = "Benchmark: Applied"
-			benchmarkStatus.Color = utils.MustHex("#4CE870")
-			benchmarkStatus.Refresh()
-			benchmarkApplyBtn.Disable()
-		})
-		if applied {
-			benchmarkApplyBtn.Disable()
-		} else {
+			benchmarkApplyBtn = widget.NewButton("Apply Benchmark", func() {
+				state.applyBenchmarkRecommendation(encoder, preset)
+				benchmarkStatus.Text = "Benchmark: Applied"
+				benchmarkStatus.Color = utils.MustHex("#4CE870")
+				benchmarkStatus.Refresh()
+				benchmarkApplyBtn.Disable()
+			})
 			benchmarkApplyBtn.Importance = widget.MediumImportance
-		}
 
-		benchmarkIndicator = container.NewHBox(benchmarkStatus, benchmarkApplyBtn)
+			benchmarkIndicator = container.NewHBox(benchmarkStatus, benchmarkApplyBtn)
+		}
 	}
 
 	backBar := ui.TintedBar(convertColor, container.NewHBox(