refactor(ui): show green $ in money column headers, bare values in cells

Money columns now always display a green "$" in the header and bare
numbers (no $ prefix) in cell values. FormatCentsBare formats cents
without the dollar sign for table cells; FormatCents (with $) is
still used in forms and the dashboard spending line.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: cpcloud

internal/app/detail_test.go

@@ -254,7 +254,7 @@ func newTestModelWithDetailRows() *Model {
 	// Seed a couple rows.
 	tab.Table.SetRows([]table.Row{
 		{"1", "2026-01-15", "Self", "", "first"},
-		{"2", "2026-02-01", "Acme", "$150.00", "second"},
+		{"2", "2026-02-01", "Acme", "150.00", "second"},
 	})
 	tab.Rows = []rowMeta{{ID: 1}, {ID: 2}}
 	tab.CellRows = [][]cell{
@@ -269,7 +269,7 @@ func newTestModelWithDetailRows() *Model {
 			{Value: "2", Kind: cellReadonly},
 			{Value: "2026-02-01", Kind: cellDate},
 			{Value: "Acme", Kind: cellText},
-			{Value: "$150.00", Kind: cellMoney},
+			{Value: "150.00", Kind: cellMoney},
 			{Value: "second", Kind: cellText},
 		},
 	}

internal/app/mag.go

@@ -100,10 +100,10 @@ func magTransformCells(rows [][]cell) [][]cell {
 	return out
 }
 
-// magAnnotateSpecs returns a copy of specs with a styled "$" suffix on
-// money column titles so the unit is visible in the header instead of
-// repeated in every cell.
-func magAnnotateSpecs(specs []columnSpec, styles Styles) []columnSpec {
+// annotateMoneyHeaders returns a copy of specs with a styled "$" suffix
+// on money column titles. The unit lives in the header so cell values
+// can be bare numbers.
+func annotateMoneyHeaders(specs []columnSpec, styles Styles) []columnSpec {
 	out := make([]columnSpec, len(specs))
 	copy(out, specs)
 	for i, spec := range out {

internal/app/mag_test.go

@@ -11,6 +11,7 @@ import (
 )
 
 func TestMagFormatMoneyWithUnit(t *testing.T) {
+	// Used by magCents for dashboard (input still has $ from FormatCents).
 	tests := []struct {
 		name  string
 		value string
@@ -31,21 +32,21 @@ func TestMagFormatMoneyWithUnit(t *testing.T) {
 	}
 }
 
-func TestMagFormatMoneyWithoutUnit(t *testing.T) {
+func TestMagFormatBareMoney(t *testing.T) {
+	// Table cells use FormatCentsBare (no $ prefix).
 	tests := []struct {
 		name  string
 		value string
 		want  string
 	}{
-		{"thousands", "$5,234.23", "\U0001F8213"},
-		{"hundreds", "$500.00", "\U0001F8212"},
-		{"millions", "$1,000,000.00", "\U0001F8216"},
-		{"tens", "$42.00", "\U0001F8211"},
-		{"single digit", "$7.50", "\U0001F8210"},
-		{"sub-dollar", "$0.50", "\U0001F821-1"},
-		{"zero", "$0.00", "\U0001F8210"},
-		{"negative", "-$5.00", "-\U0001F8210"},
-		{"negative large", "-$12,345.00", "-\U0001F8214"},
+		{"thousands", "5,234.23", "\U0001F8213"},
+		{"hundreds", "500.00", "\U0001F8212"},
+		{"millions", "1,000,000.00", "\U0001F8216"},
+		{"tens", "42.00", "\U0001F8211"},
+		{"single digit", "7.50", "\U0001F8210"},
+		{"sub-dollar", "0.50", "\U0001F821-1"},
+		{"zero", "0.00", "\U0001F8210"},
+		{"negative", "-5.00", "-\U0001F8210"},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -102,18 +103,18 @@ func TestMagFormatSkipsNonNumeric(t *testing.T) {
 	}
 }
 
-func TestMagTransformCellsStripsUnit(t *testing.T) {
+func TestMagTransformCells(t *testing.T) {
 	rows := [][]cell{
 		{
 			{Value: "1", Kind: cellReadonly},
 			{Value: "Kitchen Remodel", Kind: cellText},
-			{Value: "$5,234.23", Kind: cellMoney},
+			{Value: "5,234.23", Kind: cellMoney},
 			{Value: "3", Kind: cellDrilldown},
 		},
 		{
 			{Value: "2", Kind: cellReadonly},
 			{Value: "Deck", Kind: cellText},
-			{Value: "$100.00", Kind: cellMoney},
+			{Value: "100.00", Kind: cellMoney},
 			{Value: "0", Kind: cellDrilldown},
 		},
 	}
@@ -127,7 +128,7 @@ func TestMagTransformCellsStripsUnit(t *testing.T) {
 	assert.Equal(t, "Kitchen Remodel", out[0][1].Value)
 	assert.Equal(t, "Deck", out[1][1].Value)
 
-	// Money cells: magnitude only, no $ prefix.
+	// Money cells: magnitude only.
 	assert.Equal(t, "\U0001F8213", out[0][2].Value)
 	assert.Equal(t, "\U0001F8212", out[1][2].Value)
 
@@ -136,18 +137,18 @@ func TestMagTransformCellsStripsUnit(t *testing.T) {
 	assert.Equal(t, "\U0001F8210", out[1][3].Value)
 
 	// Original rows are not modified.
-	assert.Equal(t, "$5,234.23", rows[0][2].Value)
+	assert.Equal(t, "5,234.23", rows[0][2].Value)
 }
 
-func TestMagAnnotateSpecs(t *testing.T) {
+func TestAnnotateMoneyHeaders(t *testing.T) {
 	styles := DefaultStyles()
 	specs := []columnSpec{
 		{Title: "Name", Kind: cellText},
 		{Title: "Total", Kind: cellMoney},
 		{Title: "Labor", Kind: cellMoney},
 		{Title: "ID", Kind: cellReadonly},
 	}
-	out := magAnnotateSpecs(specs, styles)
+	out := annotateMoneyHeaders(specs, styles)
 
 	// Non-money columns unchanged.
 	assert.Equal(t, "Name", out[0].Title)
@@ -163,13 +164,13 @@ func TestMagAnnotateSpecs(t *testing.T) {
 	assert.Equal(t, "Total", specs[1].Title)
 }
 
-func TestMagAnnotateSpecsPreservesLength(t *testing.T) {
+func TestAnnotateMoneyHeadersPreservesLength(t *testing.T) {
 	styles := DefaultStyles()
 	specs := []columnSpec{
 		{Title: "A", Kind: cellText},
 		{Title: "B", Kind: cellMoney},
 	}
-	out := magAnnotateSpecs(specs, styles)
+	out := annotateMoneyHeaders(specs, styles)
 	require.Len(t, out, 2)
 }
 

internal/app/notes_test.go

@@ -22,7 +22,7 @@ func TestNotePreviewOpensOnEnter(t *testing.T) {
 	// Seed a row with a note.
 	tab.Table.SetRows(
 		[]table.Row{
-			{"1", "2026-01-15", "Self", "$50.00", "Changed the filter and checked pressure"},
+			{"1", "2026-01-15", "Self", "50.00", "Changed the filter and checked pressure"},
 		},
 	)
 	tab.Rows = []rowMeta{{ID: 1}}
@@ -31,7 +31,7 @@ func TestNotePreviewOpensOnEnter(t *testing.T) {
 			{Value: "1", Kind: cellReadonly},
 			{Value: "2026-01-15", Kind: cellDate},
 			{Value: "Self", Kind: cellText},
-			{Value: "$50.00", Kind: cellMoney},
+			{Value: "50.00", Kind: cellMoney},
 			{Value: "Changed the filter and checked pressure", Kind: cellNotes},
 		},
 	}

internal/app/rows_test.go

@@ -23,7 +23,7 @@ func TestDateValue(t *testing.T) {
 func TestCentsValue(t *testing.T) {
 	assert.Empty(t, centsValue(nil))
 	c := int64(123456)
-	assert.Equal(t, "$1,234.56", centsValue(&c))
+	assert.Equal(t, "1,234.56", centsValue(&c))
 }
 
 func TestProjectRows(t *testing.T) {
@@ -45,7 +45,7 @@ func TestProjectRows(t *testing.T) {
 	assert.Equal(t, uint(1), meta[0].ID)
 	assert.False(t, meta[0].Deleted)
 	assert.Equal(t, "Kitchen", cells[0][2].Value)
-	assert.Equal(t, "$1,000.00", cells[0][4].Value)
+	assert.Equal(t, "1,000.00", cells[0][4].Value)
 	assert.Equal(t, "2025-03-01", cells[0][6].Value)
 	assert.Equal(t, "Kitchen", rows[0][2])
 }
@@ -81,7 +81,7 @@ func TestQuoteRows(t *testing.T) {
 	assert.Equal(t, "Kitchen", cells[0][1].Value)
 	assert.Equal(t, uint(1), cells[0][1].LinkID)
 	assert.Equal(t, "ContractorCo", cells[0][2].Value)
-	assert.Equal(t, "$500.00", cells[0][3].Value)
+	assert.Equal(t, "500.00", cells[0][3].Value)
 }
 
 func TestQuoteRowsFallbackProjectName(t *testing.T) {
@@ -153,7 +153,7 @@ func TestApplianceRows(t *testing.T) {
 	assert.Equal(t, "Samsung", cells[0][2].Value)
 	assert.Equal(t, "2023-06-15", cells[0][6].Value)
 	assert.Equal(t, "2y", cells[0][7].Value)
-	assert.Equal(t, "$899.00", cells[0][9].Value)
+	assert.Equal(t, "899.00", cells[0][9].Value)
 	assert.Equal(t, "2", cells[0][10].Value)
 }
 

internal/app/sort_test.go

@@ -23,19 +23,19 @@ func newSortTab() *Tab {
 			{
 				{Value: "3", Kind: cellReadonly},
 				{Value: "Charlie", Kind: cellText},
-				{Value: "$200.00", Kind: cellMoney},
+				{Value: "200.00", Kind: cellMoney},
 				{Value: "2025-03-01", Kind: cellDate},
 			},
 			{
 				{Value: "1", Kind: cellReadonly},
 				{Value: "Alice", Kind: cellText},
-				{Value: "$50.00", Kind: cellMoney},
+				{Value: "50.00", Kind: cellMoney},
 				{Value: "2025-01-15", Kind: cellDate},
 			},
 			{
 				{Value: "2", Kind: cellReadonly},
 				{Value: "Bob", Kind: cellText},
-				{Value: "$1,000.00", Kind: cellMoney},
+				{Value: "1,000.00", Kind: cellMoney},
 				{Value: "2025-02-10", Kind: cellDate},
 			},
 		},
@@ -123,7 +123,7 @@ func TestApplySortsByMoneyAsc(t *testing.T) {
 	applySorts(tab)
 
 	costs := collectCol(tab, 2)
-	assert.Equal(t, []string{"$50.00", "$200.00", "$1,000.00"}, costs)
+	assert.Equal(t, []string{"50.00", "200.00", "1,000.00"}, costs)
 }
 
 func TestApplySortsByDateDesc(t *testing.T) {

internal/app/tables.go

@@ -434,7 +434,7 @@ func quoteRows(
 				{Value: fmt.Sprintf("%d", q.ID), Kind: cellReadonly},
 				{Value: projectName, Kind: cellText, LinkID: q.ProjectID},
 				{Value: q.Vendor.Name, Kind: cellText, LinkID: q.VendorID},
-				{Value: data.FormatCents(q.TotalCents), Kind: cellMoney},
+				{Value: data.FormatCentsBare(q.TotalCents), Kind: cellMoney},
 				{Value: centsValue(q.LaborCents), Kind: cellMoney},
 				{Value: centsValue(q.MaterialsCents), Kind: cellMoney},
 				{Value: centsValue(q.OtherCents), Kind: cellMoney},
@@ -628,7 +628,7 @@ func centsValue(cents *int64) string {
 	if cents == nil {
 		return ""
 	}
-	return data.FormatCents(*cents)
+	return data.FormatCentsBare(*cents)
 }
 
 func dateValue(value *time.Time) string {

internal/data/validation.go

@@ -53,6 +53,19 @@ func FormatCents(cents int64) string {
 	return fmt.Sprintf("%s$%s.%02d", sign, humanize.Comma(dollars), remainder)
 }
 
+// FormatCentsBare formats cents without the dollar sign (e.g. "1,234.56").
+// Used for table cells where the column header carries the unit.
+func FormatCentsBare(cents int64) string {
+	sign := ""
+	if cents < 0 {
+		sign = "-"
+		cents = -cents
+	}
+	dollars := cents / 100
+	remainder := cents % 100
+	return fmt.Sprintf("%s%s.%02d", sign, formatWithCommas(dollars), remainder)
+}
+
 func FormatOptionalCents(cents *int64) string {
 	if cents == nil {
 		return ""

internal/data/validation_test.go

@@ -52,6 +52,12 @@ func TestFormatCents(t *testing.T) {
 	assert.Equal(t, "$1,234.56", FormatCents(123456))
 }
 
+func TestFormatCentsBare(t *testing.T) {
+	assert.Equal(t, "1,234.56", FormatCentsBare(123456))
+	assert.Equal(t, "-5.00", FormatCentsBare(-500))
+	assert.Equal(t, "0.00", FormatCentsBare(0))
+}
+
 func TestParseOptionalDate(t *testing.T) {
 	date, err := ParseOptionalDate("2025-06-11")
 	require.NoError(t, err)