Skip to content

Commit a232d08

Browse files
authored
Merge pull request #1425 from krissetto/change-thinking-dropdown-symbol
Change triangle to [+] and [-] on thinking dropdown toggle
2 parents 003cc40 + ce2f302 commit a232d08

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

pkg/tui/components/reasoningblock/reasoningblock.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -402,13 +402,13 @@ func (m *Model) hasExtraContent() bool {
402402
func (m *Model) renderHeader(expanded bool) string {
403403
badge := styles.ThinkingBadgeStyle.Render("Thinking")
404404

405-
// Use filled triangles indicating action: ▲ collapse (when expanded), ▼ expand (when collapsed)
405+
// Use [+] to expand and [-] to collapse
406406
var indicator string
407407
switch {
408408
case expanded:
409-
indicator = styles.MutedStyle.Render(" ")
409+
indicator = styles.MutedStyle.Bold(true).Render(" [-]")
410410
case m.hasExtraContent():
411-
indicator = styles.MutedStyle.Render(" ")
411+
indicator = styles.MutedStyle.Bold(true).Render(" [+]")
412412
}
413413

414414
// Add tool count indicator when collapsed

pkg/tui/components/reasoningblock/reasoningblock_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ func TestReasoningBlockCollapsedWithLongContent(t *testing.T) {
5959
view := block.View()
6060
stripped := ansi.Strip(view)
6161

62-
// Should contain "Thinking" header with expand indicator ()
63-
assert.Contains(t, stripped, "Thinking ")
62+
// Should contain "Thinking" header with expand indicator ([+])
63+
assert.Contains(t, stripped, "Thinking [+]")
6464
}
6565

6666
func TestReasoningBlockExpanded(t *testing.T) {
@@ -80,8 +80,8 @@ func TestReasoningBlockExpanded(t *testing.T) {
8080
view := block.View()
8181
stripped := ansi.Strip(view)
8282

83-
// Should contain "Thinking" header with collapse indicator ()
84-
assert.Contains(t, stripped, "Thinking ")
83+
// Should contain "Thinking" header with collapse indicator ([-])
84+
assert.Contains(t, stripped, "Thinking [-]")
8585
// Should show all lines
8686
assert.Contains(t, stripped, "Line 1")
8787
assert.Contains(t, stripped, "Line 6")

0 commit comments

Comments
 (0)