fix(ui): improve logo branding and tagline centering

- render ARCHUP logo in full blue (removed two-tone split)
- fix tagline centering using lipgloss.Width instead of len
- lipgloss.Width properly handles ANSI color codes for accurate centering
- removed unused splitPos constant

commit: 539e6603fb4e673a44a107bd315a69da6fdfe5bb
author: bnema · 12mo ago · 2025-10-26 11:37 UTC
parent: 62374aa
signature: unsigned

2 files · +7 -21

M internal/ui/logo.go +3 -19


@@ -10,15 +10,12 @@ import (
10	10
11	11		const (
12	12		// Logo ASCII art (from logo.txt)
13		-	// Split at column 34 for two-tone color scheme (cyan/white)
14	13		logoASCII = ` ░█████╗░██████╗░░█████╗░██╗░░██╗██╗░░░██╗██████╗░
15	14		██╔══██╗██╔══██╗██╔══██╗██║░░██║██║░░░██║██╔══██╗
16	15		███████║██████╔╝██║░░╚═╝███████║██║░░░██║██████╔╝
17	16		██╔══██║██╔══██╗██║░░██╗██╔══██║██║░░░██║██╔═══╝░
18	17		██║░░██║██║░░██║╚█████╔╝██║░░██║╚██████╔╝██║░░░░░
19	18		╚═╝░░╚═╝╚═╝░░╚═╝░╚════╝░╚═╝░░╚═╝░╚═════╝░╚═╝░░░░░`
20		-
21		-	splitPos = 34 // Column where color split occurs
22	19		)
23	20
24	21		// Logo represents the ArchUp logo display

@@ -50,7 +47,7 @@ func NewLogo(version string) *Logo {
50	47		}
51	48		}
52	49
53		-	// Render renders the logo with two-tone coloring (cyan/white split)
	50	+	// Render renders the logo in full blue
54	51		func (l *Logo) Render() string {
55	52		lines := strings.Split(logoASCII, "\n")
56	53		var rendered strings.Builder

@@ -61,21 +58,8 @@ func (l *Logo) Render() string {
61	58		continue
62	59		}
63	60
64		-	// Split line at splitPos for two-tone effect
65		-	var left, right string
66		-	if len(line) > splitPos {
67		-	left = line[:splitPos]
68		-	right = line[splitPos:]
69		-	} else {
70		-	left = line
71		-	right = ""
72		-	}
73		-
74		-	// Render with colors
75		-	rendered.WriteString(styles.LogoCyanStyle.Render(left))
76		-	if right != "" {
77		-	rendered.WriteString(styles.LogoWhiteStyle.Render(right))
78		-	}
	61	+	// Render entire logo in blue
	62	+	rendered.WriteString(styles.LogoCyanStyle.Render(line))
79	63		rendered.WriteString("\n")
80	64		}
81	65

 const (
 	// Logo ASCII art (from logo.txt)
-	// Split at column 34 for two-tone color scheme (cyan/white)
 	logoASCII = `  ░█████╗░██████╗░░█████╗░██╗░░██╗██╗░░░██╗██████╗░
   ██╔══██╗██╔══██╗██╔══██╗██║░░██║██║░░░██║██╔══██╗
   ███████║██████╔╝██║░░╚═╝███████║██║░░░██║██████╔╝
   ██╔══██║██╔══██╗██║░░██╗██╔══██║██║░░░██║██╔═══╝░
   ██║░░██║██║░░██║╚█████╔╝██║░░██║╚██████╔╝██║░░░░░
   ╚═╝░░╚═╝╚═╝░░╚═╝░╚════╝░╚═╝░░╚═╝░╚═════╝░╚═╝░░░░░`
-	splitPos = 34 // Column where color split occurs
 )
 // Logo represents the ArchUp logo display
 	}
 }
-// Render renders the logo with two-tone coloring (cyan/white split)
+// Render renders the logo in full blue
 func (l *Logo) Render() string {
 	lines := strings.Split(logoASCII, "\n")
 	var rendered strings.Builder
 			continue
 		}
-		// Split line at splitPos for two-tone effect
+		// Render entire logo in blue
-		var left, right string
+		rendered.WriteString(styles.LogoCyanStyle.Render(line))
-		if len(line) > splitPos {
-			left = line[:splitPos]
-			right = line[splitPos:]
-		} else {
-			left = line
-			right = ""
-		}
-		// Render with colors
-		rendered.WriteString(styles.LogoCyanStyle.Render(left))
-		if right != "" {
-			rendered.WriteString(styles.LogoWhiteStyle.Render(right))
-		}
 		rendered.WriteString("\n")
 	}

M internal/ui/model.go +4 -2


@@ -2,9 +2,11 @@ package ui
2	2
3	3		import (
4	4		"fmt"
	5	+	"strings"
5	6
6	7		tea "github.com/charmbracelet/bubbletea"
7	8		"github.com/charmbracelet/huh"
	9	+	"github.com/charmbracelet/lipgloss"
8	10		"github.com/bnema/archup/internal/config"
9	11		"github.com/bnema/archup/internal/phases"
10	12		"github.com/bnema/archup/internal/ui/components"

@@ -245,11 +247,11 @@ type phaseCompleteMsg struct {
245	247
246	248		// centerText centers text within a given width
247	249		func centerText(text string, width int) string {
248		-	textLen := len(text)
	250	+	textLen := lipgloss.Width(text)
249	251		switch {
250	252		case textLen >= width:
251	253		return text
252	254		}
253	255		padding := (width - textLen) / 2
254		-	return fmt.Sprintf("%*s", padding+textLen, text)
	256	+	return strings.Repeat(" ", padding) + text
255	257		}

 import (
 	"fmt"
+	"strings"
 	tea "github.com/charmbracelet/bubbletea"
 	"github.com/charmbracelet/huh"
+	"github.com/charmbracelet/lipgloss"
 	"github.com/bnema/archup/internal/config"
 	"github.com/bnema/archup/internal/phases"
 	"github.com/bnema/archup/internal/ui/components"
 // centerText centers text within a given width
 func centerText(text string, width int) string {
-	textLen := len(text)
+	textLen := lipgloss.Width(text)
 	switch {
 	case textLen >= width:
 		return text
 	}
 	padding := (width - textLen) / 2
-	return fmt.Sprintf("%*s", padding+textLen, text)
+	return strings.Repeat(" ", padding) + text
 }