require origin remote

Author: karolzwolak

pkg/gui/controllers/remotes_controller.go

@@ -76,7 +76,7 @@ func (self *RemotesController) GetKeybindings(opts types.KeybindingsOpts) []*typ
 		{
 			Key:               opts.GetKey(opts.Config.Branches.AddForkRemote),
 			Handler:           self.addFork,
-			GetDisabledReason: self.require(self.singleItemSelected()),
+			GetDisabledReason: self.hasOriginRemote(),
 			Description:       self.c.Tr.AddForkRemote,
 			Tooltip:           self.c.Tr.AddForkRemoteTooltip,
 			DisplayOnScreen:   true,
@@ -244,6 +244,16 @@ func (self *RemotesController) getOrigin() *models.Remote {
 	return nil
 }
 
+func (self *RemotesController) hasOriginRemote() func() *types.DisabledReason {
+	return func() *types.DisabledReason {
+		if self.getOrigin() == nil {
+			return &types.DisabledReason{Text: self.c.Tr.NoOriginRemote}
+		}
+
+		return nil
+	}
+}
+
 func (self *RemotesController) addFork() error {
 	baseRemote := self.getOrigin()
 

pkg/i18n/english.go

@@ -529,6 +529,7 @@ type TranslationSet struct {
 	AddForkRemoteUsername                 string
 	AddForkRemoteTooltip                  string
 	IncompatibleForkAlreadyExistsError    string
+	NoOriginRemote                        string
 	ViewBranches                          string
 	EditRemoteName                        string
 	EditRemoteUrl                         string
@@ -1630,6 +1631,7 @@ func EnglishTranslationSet() *TranslationSet {
 		AddForkRemote:                        `Add fork remote`,
 		AddForkRemoteTooltip:                 `Quickly add a fork remote by replacing the owner in the origin URL and optionally check out a branch from new remote.`,
 		IncompatibleForkAlreadyExistsError:   `Remote {{.remoteName}} already exists and has different URL`,
+		NoOriginRemote:                       "Action needs 'origin' remote",
 		ViewBranches:                         "View branches",
 		EditRemoteName:                       `Enter updated remote name for {{.remoteName}}:`,
 		EditRemoteUrl:                        `Enter updated remote url for {{.remoteName}}:`,