Better control of checked nodes

StanleyGoldman · StanleyGoldman · commit 9bed325ef5ff · 2017-12-07T18:04:24.000-05:00
diff --git a/src/GitHub.Api/UI/TreeBase.cs b/src/GitHub.Api/UI/TreeBase.cs
@@ -109,13 +109,17 @@ public void CheckAll(bool isChecked)
             {
                 var wasChecked = node.CheckState == CheckState.Checked;
                 node.CheckState = nodeCheckState;
-                if (isChecked && !wasChecked)
-                {
-                    AddCheckedNode(node);
-                }
-                else if (!isChecked && wasChecked)
+
+                if (!node.IsFolder)
                 {
-                    RemoveCheckedNode(node);
+                    if (isChecked && !wasChecked)
+                    {
+                        AddCheckedNode(node);
+                    }
+                    else if (!isChecked && wasChecked)
+                    {
+                        RemoveCheckedNode(node);
+                    }
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -109,13 +109,17 @@ public void CheckAll(bool isChecked)`
`109`	`109`	`{`
`110`	`110`	`var wasChecked = node.CheckState == CheckState.Checked;`
`111`	`111`	`node.CheckState = nodeCheckState;`
`112`		`- if (isChecked && !wasChecked)`
`113`		`- {`
`114`		`- AddCheckedNode(node);`
`115`		`- }`
`116`		`- else if (!isChecked && wasChecked)`
	`112`	`+`
	`113`	`+ if (!node.IsFolder)`
`117`	`114`	`{`
`118`		`- RemoveCheckedNode(node);`
	`115`	`+ if (isChecked && !wasChecked)`
	`116`	`+ {`
	`117`	`+ AddCheckedNode(node);`
	`118`	`+ }`
	`119`	`+ else if (!isChecked && wasChecked)`
	`120`	`+ {`
	`121`	`+ RemoveCheckedNode(node);`
	`122`	`+ }`
`119`	`123`	`}`
`120`	`124`	`}`
`121`	`125`	`}`