Skip to content

Commit

Permalink
Add ability to configure branch color patterns
Browse files Browse the repository at this point in the history
  • Loading branch information
mtrajano committed Jan 5, 2025
1 parent ef718f3 commit 1834be5
Show file tree
Hide file tree
Showing 4 changed files with 50 additions and 18 deletions.
7 changes: 7 additions & 0 deletions docs/Config.md
Original file line number Diff line number Diff line change
Expand Up @@ -836,8 +836,15 @@ You can customize the color of branches based on the branch prefix:

```yaml
gui:
# NOTE: that this configuration will be deprecated in favor of using branchColorPatterns below
branchColors:
'docs': '#11aaff' # use a light blue for branches beginning with 'docs/'
# alternatively you can use a regex pattern as your coloring rules
# NOTE: this configuration overwrites the one above, if you would like to set a similar rule see the example below
branchColorPatterns:
'docs/.+': '#11aaff' # similar to the previous configuration above, setting branches that begin with 'docs/'
'ISSUE-\d+': '#ff5733' # use a bright orange for branches beginning with 'ISSUE-'
```

## Example Coloring
Expand Down
4 changes: 3 additions & 1 deletion pkg/config/user_config.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,9 @@ type GuiConfig struct {
// See https://github.com/jesseduffield/lazygit/blob/master/docs/Config.md#custom-author-color
AuthorColors map[string]string `yaml:"authorColors"`
// See https://github.com/jesseduffield/lazygit/blob/master/docs/Config.md#custom-branch-color
BranchColors map[string]string `yaml:"branchColors"`
// NOTE: BranchColors is being deprecated in favor of BranchColorPatterns
BranchColors map[string]string `yaml:"branchColors"`
BranchColorPatterns map[string]string `yaml:"branchColorPatterns"`
// The number of lines you scroll by when scrolling the main window
ScrollHeight int `yaml:"scrollHeight" jsonschema:"minimum=1"`
// If true, allow scrolling past the bottom of the content in the main window
Expand Down
8 changes: 7 additions & 1 deletion pkg/gui/gui.go
Original file line number Diff line number Diff line change
Expand Up @@ -454,7 +454,13 @@ func (gui *Gui) onUserConfigLoaded() error {
} else if userConfig.Gui.ShowIcons {
icons.SetNerdFontsVersion("2")
}
presentation.SetCustomBranches(userConfig.Gui.BranchColors)

if len(userConfig.Gui.BranchColorPatterns) > 0 {
presentation.SetCustomBranches(userConfig.Gui.BranchColorPatterns, true)
} else {
// The alternative is to match on branch types with the branchColors config which will be deprecated in the future
presentation.SetCustomBranches(userConfig.Gui.BranchColors, false)
}

return nil
}
Expand Down
49 changes: 33 additions & 16 deletions pkg/gui/presentation/branches.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ package presentation

import (
"fmt"
"regexp"
"strings"
"time"

Expand All @@ -18,7 +19,12 @@ import (
"github.com/samber/lo"
)

var branchPrefixColorCache = make(map[string]style.TextStyle)
type colorMatcher struct {
patterns map[string]style.TextStyle
isRegex bool // NOTE: this value is needed only until branchColors is deprecated and only regex color patterns are used
}

var colorPatterns *colorMatcher

func GetBranchListDisplayStrings(
branches []*models.Branch,
Expand Down Expand Up @@ -125,22 +131,30 @@ func getBranchDisplayStrings(

// GetBranchTextStyle branch color
func GetBranchTextStyle(name string) style.TextStyle {
branchType := strings.Split(name, "/")[0]

if value, ok := branchPrefixColorCache[branchType]; ok {
return value
if style, ok := colorPatterns.match(name); ok {
return *style
}

switch branchType {
case "feature":
return style.FgGreen
case "bugfix":
return style.FgYellow
case "hotfix":
return style.FgRed
default:
return theme.DefaultTextColor
// if no match return the default
return theme.DefaultTextColor
}

func (m *colorMatcher) match(name string) (*style.TextStyle, bool) {
if m.isRegex {
for pattern, style := range m.patterns {
if matched, _ := regexp.MatchString(pattern, name); matched {
return &style, true
}
}
} else {
// old behavior using the deprecated branchColors behavior matching on branch type
branchType := strings.Split(name, "/")[0]
if value, ok := m.patterns[branchType]; ok {
return &value, true
}
}

return nil, false
}

func BranchStatus(
Expand Down Expand Up @@ -189,6 +203,9 @@ func BranchStatus(
return result
}

func SetCustomBranches(customBranchColors map[string]string) {
branchPrefixColorCache = utils.SetCustomColors(customBranchColors)
func SetCustomBranches(customBranchColors map[string]string, isRegex bool) {
colorPatterns = &colorMatcher{
patterns: utils.SetCustomColors(customBranchColors),
isRegex: isRegex,
}
}

0 comments on commit 1834be5

Please sign in to comment.