Skip to content

Commit f765fdf

Browse files
wesmclaude
andcommitted
fix: re-dirty column options on save failure and fix modal help text
Re-set colOptionsDirty on configSaveErrMsg so closing the modal again retries persistence without requiring the user to toggle something first. Show "borders" instead of "toggle" in the column options help when opened from the tasks view, where space only toggles borders (column visibility is not available). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 15b882e commit f765fdf

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

cmd/roborev/tui/render_queue.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -851,8 +851,12 @@ func (m model) renderColumnOptionsView() string {
851851
}
852852

853853
b.WriteString("\n")
854+
toggleHelp := helpItem{"space", "toggle"}
855+
if m.colOptionsReturnView == viewTasks {
856+
toggleHelp = helpItem{"space", "borders"}
857+
}
854858
helpRows := [][]helpItem{
855-
{{"↑/↓", "navigate"}, {"j/k", "reorder"}, {"space", "toggle"}, {"esc", "close"}},
859+
{{"↑/↓", "navigate"}, {"j/k", "reorder"}, toggleHelp, {"esc", "close"}},
856860
}
857861
b.WriteString(renderHelpTable(helpRows, m.width))
858862

cmd/roborev/tui/tui.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -661,6 +661,7 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
661661
case applyPatchResultMsg:
662662
return m.handleApplyPatchResultMsg(msg)
663663
case configSaveErrMsg:
664+
m.colOptionsDirty = true
664665
m.setFlash("Config save failed: "+msg.err.Error(), 5*time.Second, m.currentView)
665666
return m, nil
666667
}

0 commit comments

Comments
 (0)