Skip to content

Commit

Permalink
revert(git): remove trailing space from upstream icons
Browse files Browse the repository at this point in the history
This reverts commit f277c19 as it was
not the correct change to make. The upstream icons are supposed to
have trailing spaces so they display correctly in the default template.

resolves #4784
  • Loading branch information
JanDeDobbeleer committed Mar 19, 2024
1 parent 6bbdfb9 commit cd232cd
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 15 deletions.
14 changes: 7 additions & 7 deletions src/segments/git.go
Expand Up @@ -492,19 +492,19 @@ func (g *Git) getUpstreamIcon() string {
Icon properties.Property
Default string
}{
"github": {GithubIcon, "\uF408"},
"gitlab": {GitlabIcon, "\uF296"},
"bitbucket": {BitbucketIcon, "\uF171"},
"dev.azure.com": {AzureDevOpsIcon, "\uEBE8"},
"visualstudio.com": {AzureDevOpsIcon, "\uEBE8"},
"codecommit": {CodeCommit, "\uF270"},
"github": {GithubIcon, "\uF408 "},
"gitlab": {GitlabIcon, "\uF296 "},
"bitbucket": {BitbucketIcon, "\uF171 "},
"dev.azure.com": {AzureDevOpsIcon, "\uEBE8 "},
"visualstudio.com": {AzureDevOpsIcon, "\uEBE8 "},
"codecommit": {CodeCommit, "\uF270 "},
}
for key, value := range defaults {
if strings.Contains(g.UpstreamURL, key) {
return g.props.GetString(value.Icon, value.Default)
}
}
return g.props.GetString(GitIcon, "\uE5FB")
return g.props.GetString(GitIcon, "\uE5FB ")
}

func (g *Git) setGitStatus() {
Expand Down
2 changes: 1 addition & 1 deletion src/segments/git_test.go
Expand Up @@ -183,7 +183,7 @@ func TestEnabledInBareRepo(t *testing.T) {
FetchRemote: true,
Remote: "origin",
RemoteURL: "git@github.com:JanDeDobbeleer/oh-my-posh.git",
ExpectedRemote: "\uf408",
ExpectedRemote: "\uf408 ",
},
}
for _, tc := range cases {
Expand Down
14 changes: 7 additions & 7 deletions website/docs/segments/git.mdx
Expand Up @@ -111,12 +111,12 @@ You can set the following properties to `true` to enable fetching additional inf

| Name | Type | Description |
| ------------------- | ------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| `github_icon` | `string` | icon/text to display when the upstream is Github - defaults to `\uF408` |
| `gitlab_icon` | `string` | icon/text to display when the upstream is Gitlab - defaults to `\uF296` |
| `bitbucket_icon` | `string` | icon/text to display when the upstream is Bitbucket - defaults to `\uF171` |
| `azure_devops_icon` | `string` | icon/text to display when the upstream is Azure DevOps - defaults to `\uEBE8` |
| `codecommit_icon` | `string` | icon/text to display when the upstream is AWS CodeCommit - defaults to `\uF270` |
| `git_icon` | `string` | icon/text to display when the upstream is not known/mapped - defaults to `\uE5FB` |
| `github_icon` | `string` | icon/text to display when the upstream is Github - defaults to `\uF408 ` |
| `gitlab_icon` | `string` | icon/text to display when the upstream is Gitlab - defaults to `\uF296 ` |
| `bitbucket_icon` | `string` | icon/text to display when the upstream is Bitbucket - defaults to `\uF171 ` |
| `azure_devops_icon` | `string` | icon/text to display when the upstream is Azure DevOps - defaults to `\uEBE8 ` |
| `codecommit_icon` | `string` | icon/text to display when the upstream is AWS CodeCommit - defaults to `\uF270 ` |
| `git_icon` | `string` | icon/text to display when the upstream is not known/mapped - defaults to `\uE5FB ` |
| `upstream_icons` | `map[string]string` | a key, value map representing the remote URL (or a part of that URL) and icon to use in case the upstream URL contains the key. These get precedence over the standard icons |

## Template ([info][templates])
Expand Down Expand Up @@ -157,7 +157,7 @@ You can set the following properties to `true` to enable fetching additional inf
| `.Rebase` | `boolean` | true when in a rebase |
| `.CherryPick` | `boolean` | true when in a cherry pick |
| `.Revert` | `boolean` | true when in a revert |
| `.LatestTag` | `string` | the latest tag name |
| `.LatestTag` | `string` | the latest tag name |

### Status

Expand Down

0 comments on commit cd232cd

Please sign in to comment.