feat(ui): update the PR view diff changes summary

[Sanix-Darker]

Summary

UI changes proposal on +/- with colors like the view we have on github.

• update colors (green/red) for adds/deletions.
• add a formater util function for killos /Millions changes diffs.
• add a sync amount of spaces between adds and deletions.

How did you test this change?

On my Ubuntu Os 20.04 , tmux, under allacrity.

go run gh-dash.go --debug

Images/Videos

BEFORE

AFTER

NOTE

• This should work for all Unix based system.
• I didn't create an issue because i thought there were not too many things to do on this branch, I hope it's okay.

[dlvhdr]

This is great thank you! <3

[dlvhdr]

You should use lipgloss for this. Specifically lipgloss.NewStyle().Foreground(lipgloss.Color("1")) and then use lipgloss.JoinHorizontal()

[Sanix-Darker]

Thanks for the review @dlvhdr
i updated the method like this :

func (pr *PullRequest) renderLines() string {
	deletions := 0
	if pr.Data.Deletions > 0 {
		deletions = pr.Data.Deletions
	}

    greenStyle := lipgloss.NewStyle().Foreground(lipgloss.Color("2"))
    redStyle := lipgloss.NewStyle().Foreground(lipgloss.Color("1"))

    greenText := greenStyle.Render("+" + components.FormatNumber(pr.Data.Additions))
    redText := redStyle.Render("-" + components.FormatNumber(deletions))

    return pr.getTextStyle().Render(
        components.KeepSameSpacesOnAddDeletions(
            lipgloss.JoinHorizontal(lipgloss.Top, greenText, redText),
        ),
    )
}

but am having an output not "fine" for the redText :

Any tips ? 🤔

[dlvhdr]

Not near a computer right now but look at the code, there are other instances where I applied the "selected" background again. When you create a style you lose the bg the cell previously had, so you need to reapply it.

[dlvhdr]

@Sanix-Darker I fixed the issue, also see the open issue on lipgloss for reference: charmbracelet/lipgloss#144

[Sanix-Darker]

looks amazing @dlvhdr , thank you so much !