Skip to content
Snippets Groups Projects
Commit 2fcbf68b authored by Jessica Yuen's avatar Jessica Yuen
Browse files

Use background colors for param diff

Foreground colors were causing conflicts with user's terminal colors.
i.e. Black text would show as invisible for users with a black terminal
background.
parent 9c2d0a2d
No related branches found
No related tags found
No related merge requests found
...@@ -289,9 +289,9 @@ func (c *ParamDiffCmd) Run(out io.Writer) error { ...@@ -289,9 +289,9 @@ func (c *ParamDiffCmd) Run(out io.Writer) error {
envSpacing := strings.Repeat(" ", maxEnvLen-maxParamLen-len(c.env1)) envSpacing := strings.Repeat(" ", maxEnvLen-maxParamLen-len(c.env1))
// print headers // print headers
color.New(color.FgBlack).Fprintln(out, paramComponentHeader+componentSpacing+ fmt.Fprintln(out, paramComponentHeader+componentSpacing+
paramNameHeader+nameSpacing+c.env1+envSpacing+c.env2) paramNameHeader+nameSpacing+c.env1+envSpacing+c.env2)
color.New(color.FgBlack).Fprintln(out, strings.Repeat("=", len(paramComponentHeader))+componentSpacing+ fmt.Fprintln(out, strings.Repeat("=", len(paramComponentHeader))+componentSpacing+
strings.Repeat("=", len(paramNameHeader))+nameSpacing+ strings.Repeat("=", len(paramNameHeader))+nameSpacing+
strings.Repeat("=", len(c.env1))+envSpacing+ strings.Repeat("=", len(c.env1))+envSpacing+
strings.Repeat("=", len(c.env2))) strings.Repeat("=", len(c.env2)))
...@@ -303,13 +303,16 @@ func (c *ParamDiffCmd) Run(out io.Writer) error { ...@@ -303,13 +303,16 @@ func (c *ParamDiffCmd) Run(out io.Writer) error {
envSpacing = strings.Repeat(" ", maxEnvLen-maxParamLen-len(k.value1)) envSpacing = strings.Repeat(" ", maxEnvLen-maxParamLen-len(k.value1))
line := fmt.Sprint(k.component + componentSpacing + k.param + nameSpacing + k.value1 + envSpacing + k.value2) line := fmt.Sprint(k.component + componentSpacing + k.param + nameSpacing + k.value1 + envSpacing + k.value2)
if len(k.value1) == 0 { if len(k.value1) == 0 {
color.New(color.FgGreen).Fprintln(out, line) color.New(color.BgGreen).Fprint(out, line)
fmt.Fprintln(out)
} else if len(k.value2) == 0 { } else if len(k.value2) == 0 {
color.New(color.FgRed).Fprintln(out, line) color.New(color.BgRed).Fprint(out, line)
fmt.Fprintln(out)
} else if k.value1 != k.value2 { } else if k.value1 != k.value2 {
color.New(color.FgYellow).Fprintln(out, line) color.New(color.BgYellow).Fprint(out, line)
fmt.Fprintln(out)
} else { } else {
color.New(color.FgBlack).Fprintln(out, line) fmt.Fprintln(out, line)
} }
} }
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment