From 04038e7e56e36b9868a415b96f98b29b2142636c Mon Sep 17 00:00:00 2001
From: Ayman Bagabas <ayman.bagabas@gmail.com>
Date: Wed, 5 Apr 2023 13:33:14 -0400
Subject: [PATCH] fix(ui): spinner regression & empty repo

---
 ui/pages/repo/empty.go  | 12 +++++-------
 ui/pages/repo/readme.go |  5 +----
 ui/pages/repo/refs.go   |  7 ++++---
 ui/pages/repo/repo.go   |  1 +
 ui/styles/styles.go     |  2 +-
 5 files changed, 12 insertions(+), 15 deletions(-)

diff --git a/ui/pages/repo/empty.go b/ui/pages/repo/empty.go
index d5230c42a5da1179bf4d2e5ef6e1c6abfeb83838..fe3921d9c66fe1d6619a412a33558b418fa41694 100644
--- a/ui/pages/repo/empty.go
+++ b/ui/pages/repo/empty.go
@@ -2,14 +2,12 @@ package repo
 
 import (
 	"fmt"
-	"strings"
 
 	"github.com/charmbracelet/soft-serve/server/config"
+	"github.com/charmbracelet/soft-serve/ui/common"
 )
 
 func defaultEmptyRepoMsg(cfg *config.Config, repo string) string {
-	publicURL := cfg.SSH.PublicURL
-	repo = strings.TrimSuffix(repo, ".git")
 	return fmt.Sprintf(`# Quick Start
 
 Get started by cloning this repository, add your files, commit, and push.
@@ -17,7 +15,7 @@ Get started by cloning this repository, add your files, commit, and push.
 ## Clone this repository.
 
 `+"```"+`sh
-git clone %[1]s/%[2]s.git
+git clone %[1]s
 `+"```"+`
 
 ## Creating a new repository on the command line
@@ -28,15 +26,15 @@ git init
 git add README.md
 git branch -M main
 git commit -m "first commit"
-git remote add origin %[1]s/%[2]s.git
+git remote add origin %[1]s
 git push -u origin main
 `+"```"+`
 
 ## Pushing an existing repository from the command line
 
 `+"```"+`sh
-git remote add origin %[1]s/%[2]s.git
+git remote add origin %[1]s
 git push -u origin main
 `+"```"+`
-`, publicURL, repo)
+`, common.RepoURL(cfg.SSH.PublicURL, repo))
 }
diff --git a/ui/pages/repo/readme.go b/ui/pages/repo/readme.go
index 5374779b9a1e3560b06eaf46fdb8c357dbc851b3..e10eb3c6b55603d17680969fa95b0eacb25a5560 100644
--- a/ui/pages/repo/readme.go
+++ b/ui/pages/repo/readme.go
@@ -117,10 +117,7 @@ func (r *Readme) updateReadmeCmd() tea.Msg {
 	if r.repo == nil {
 		return common.ErrorCmd(common.ErrMissingRepo)
 	}
-	rm, rp, err := backend.Readme(r.repo)
-	if err != nil {
-		return common.ErrorCmd(err)
-	}
+	rm, rp, _ := backend.Readme(r.repo)
 	r.readmePath = rp
 	r.code.GotoTop()
 	cmd := r.code.SetContent(rm, rp)
diff --git a/ui/pages/repo/refs.go b/ui/pages/repo/refs.go
index d107264e0e4304766f682f272f1ee0c3decb0585..90b05c22e6145218d6f1cd98e6b15b8832a3315d 100644
--- a/ui/pages/repo/refs.go
+++ b/ui/pages/repo/refs.go
@@ -222,11 +222,12 @@ func UpdateRefCmd(repo backend.Repository) tea.Cmd {
 		if err != nil {
 			return common.ErrorMsg(err)
 		}
+		bs, _ := r.Branches()
+		if len(bs) == 0 {
+			return EmptyRepoMsg{}
+		}
 		ref, err := r.HEAD()
 		if err != nil {
-			if bs, err := r.Branches(); err != nil && len(bs) == 0 {
-				return EmptyRepoMsg{}
-			}
 			logger.Debugf("ui: error getting HEAD reference: %v", err)
 			return common.ErrorMsg(err)
 		}
diff --git a/ui/pages/repo/repo.go b/ui/pages/repo/repo.go
index ff94c890d955ada5b2fd679619d5c0764db72c0a..6be90105feea3b7fd650919882fe99dc618598a3 100644
--- a/ui/pages/repo/repo.go
+++ b/ui/pages/repo/repo.go
@@ -180,6 +180,7 @@ func (r *Repo) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			r.tabs.Init(),
 			// This will set the selected repo in each pane's model.
 			r.updateModels(msg),
+			r.spinner.Tick,
 		)
 	case RefMsg:
 		r.ref = msg
diff --git a/ui/styles/styles.go b/ui/styles/styles.go
index cd50439cf6fa2004693296007f0f48c00f4bf435..1faac44f93d5ec404baf53f943c41b0bea4519ee 100644
--- a/ui/styles/styles.go
+++ b/ui/styles/styles.go
@@ -413,7 +413,7 @@ func DefaultStyles() *Styles {
 
 	s.NoContent = lipgloss.NewStyle().
 		SetString("No Content.").
-		MarginTop(2).
+		MarginTop(1).
 		MarginLeft(2).
 		Foreground(lipgloss.Color("242"))