fix offset parameter display

emprcl · Aug 9, 2024 · 8b9e4da · 8b9e4da
1 parent 6243b4c
commit 8b9e4da
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/sequencer/step.go b/sequencer/step.go
@@ -159,9 +159,6 @@ func (s step) ProbabilityString() string {
 
 // OffsetString returns the string representation of the step offset.
 func (s step) OffsetString() string {
-	if s.offset == 0 {
-		return ""
-	}
 	return fmt.Sprintf("+%d", s.offset)
 }
 

diff --git a/ui/step.go b/ui/step.go
@@ -108,6 +108,10 @@ func (m mainModel) renderStepContent(step sequencer.Step) string {
 	if !step.IsActive() {
 		return strconv.Itoa(step.Position() + 1)
 	}
+	offset := step.OffsetString()
+	if step.Offset() == 0 {
+		offset = ""
+	}
 	return lipgloss.JoinVertical(
 		lipgloss.Left,
 		fmt.Sprintf("%d", step.Position()+1),
@@ -120,7 +124,7 @@ func (m mainModel) renderStepContent(step sequencer.Step) string {
 				Render(step.ProbabilityString()),
 			lipgloss.NewStyle().
 				MarginLeft(1).
-				Render(step.OffsetString()),
+				Render(offset),
 		),
 	)
 }