dynamic grouping/tree fix #257

jonshaffer · 2016-12-13T16:09:59Z

Fix for #212

In demo, sets group:true to 1 column (the code only accounts for this anyways at this time)
pulled refreshTree, refreshGroup logic into its own method for calling on groupColumn || treeColumn change
added a watch on the options.columns that updates group/tree for the table
moved conditional scope.watches from the constructor to a method that gets reevaluated if columns ever change

jonshaffer · 2016-12-13T16:53:12Z

src/components/body/BodyController.js

@@ -12,74 +12,112 @@ export class BodyController{
  constructor($scope, $timeout){


can remove $timeout here && on doc

jonshaffer · 2016-12-13T16:54:47Z

src/components/body/BodyController.js

+
+        this.setTreeAndGroupColumns();
+
+        this.setConditionalWatches();


this might be more accurate be on a watch for options.scrollbarV && options.paging.externalPaging

dynamic grouping/tree fix

9ac9030

jonshaffer commented Dec 13, 2016

View reviewed changes

-useless timeout injection

3c5888d

Provide feedback

		@@ -12,74 +12,112 @@ export class BodyController{
		constructor($scope, $timeout){