Disable delete button if the selected item is a folder

StanleyGoldman · StanleyGoldman · commit de7dade71b8e · 2017-08-28T11:04:38.000-04:00
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BranchesView.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/BranchesView.cs
@@ -464,7 +464,7 @@ private void OnButtonBarGUI()
             {
                 // Delete button
                 // If the current branch is selected, then do not enable the Delete button
-                var disableDelete = selectedNode == null || activeBranchNode == selectedNode;
+                var disableDelete = selectedNode == null || selectedNode.Type == NodeType.Folder || activeBranchNode == selectedNode;
                 EditorGUI.BeginDisabledGroup(disableDelete);
                 {
                     if (GUILayout.Button(DeleteBranchButton, EditorStyles.miniButton, GUILayout.ExpandWidth(false)))

Original file line number	Diff line number	Diff line change
`@@ -464,7 +464,7 @@ private void OnButtonBarGUI()`
`464`	`464`	`{`
`465`	`465`	`// Delete button`
`466`	`466`	`// If the current branch is selected, then do not enable the Delete button`
`467`		`- var disableDelete = selectedNode == null \|\| activeBranchNode == selectedNode;`
	`467`	`+ var disableDelete = selectedNode == null \|\| selectedNode.Type == NodeType.Folder \|\| activeBranchNode == selectedNode;`
`468`	`468`	`EditorGUI.BeginDisabledGroup(disableDelete);`
`469`	`469`	`{`
`470`	`470`	`if (GUILayout.Button(DeleteBranchButton, EditorStyles.miniButton, GUILayout.ExpandWidth(false)))`