Correcting how child nodes are identified

StanleyGoldman · StanleyGoldman · commit bdc1ebb4ee6f · 2017-12-06T17:25:12.000-05:00
diff --git a/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/TreeControl.cs b/src/UnityExtension/Assets/Editor/GitHub.Unity/UI/TreeControl.cs
@@ -293,14 +293,9 @@ private void ToggleNodeChecked(int idx, TreeNode node)
 
         private void ToggleChildrenChecked(int idx, TreeNode node, bool isChecked)
         {
-            for (var i = idx + 1; i < nodes.Count && node.Level <= nodes[i].Level; i++)
+            for (var i = idx + 1; i < nodes.Count && node.Level < nodes[i].Level; i++)
             {
                 var childNode = nodes[i];
-                if (node.Level == childNode.Level)
-                {
-                    continue;
-                }
-
                 childNode.CheckState = isChecked ? CheckState.Checked : CheckState.Empty;
 
                 if (childNode.IsFolder)

Original file line number	Diff line number	Diff line change
`@@ -293,14 +293,9 @@ private void ToggleNodeChecked(int idx, TreeNode node)`
`293`	`293`
`294`	`294`	`private void ToggleChildrenChecked(int idx, TreeNode node, bool isChecked)`
`295`	`295`	`{`
`296`		`- for (var i = idx + 1; i < nodes.Count && node.Level <= nodes[i].Level; i++)`
	`296`	`+ for (var i = idx + 1; i < nodes.Count && node.Level < nodes[i].Level; i++)`
`297`	`297`	`{`
`298`	`298`	`var childNode = nodes[i];`
`299`		`- if (node.Level == childNode.Level)`
`300`		`- {`
`301`		`- continue;`
`302`		`- }`
`303`		`-`
`304`	`299`	`childNode.CheckState = isChecked ? CheckState.Checked : CheckState.Empty;`
`305`	`300`
`306`	`301`	`if (childNode.IsFolder)`