Compare commits

...

2 Commits

Author SHA1 Message Date
e951f40894 Update DONE.md with benchmark UI cleanup feature
Added documentation for hiding benchmark indicator when settings are already applied.

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

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
2025-12-28 22:21:33 -05:00
8ce6240c02 Hide benchmark indicator in Convert module when already applied
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>
2025-12-28 22:21:09 -05:00
2 changed files with 22 additions and 21 deletions

View File

@ -74,6 +74,12 @@ This file tracks completed features, fixes, and milestones.
- Combined with 500ms manual throttle in refreshQueueView() for optimal balance
### User Experience Improvements
- ✅ **Benchmark UI Cleanup**
- Hide benchmark indicator in Convert module when settings are already applied
- Only show "Benchmark: Not Applied" status when action is needed
- Removes clutter from UI when using benchmark settings
- Cleaner interface for active conversions with benchmark recommendations
- ✅ **Queue Position Labeling**
- Fixed confusing priority display in queue view
- Changed from internal priority numbers (3, 2, 1) to user-friendly queue positions (1, 2, 3)

37
main.go
View File

@ -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(