From 18e0aabdb20e1f232cb507d7cb651df49aa33b41 Mon Sep 17 00:00:00 2001 From: Shannon Date: Fri, 10 Oct 2014 10:49:47 +1100 Subject: [PATCH] adds some notes --- .../common/dialogs/treepicker.controller.js | 4 ++++ .../src/views/common/dialogs/treepicker.html | 20 ------------------- 2 files changed, 4 insertions(+), 20 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.controller.js b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.controller.js index 280ee0c132..11cd5a52a1 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.controller.js +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.controller.js @@ -188,6 +188,10 @@ angular.module("umbraco").controller("Umbraco.Dialogs.TreePickerController", $scope.hideSearch = function () { + //TODO: Move this to the treeService, we don't need a reference to the 'tree' the way this is working + // because if we have a single node, that is all we need since we can traverse to the tree root in the treeService. + // this logic needs to be centralized so it can be used in other tree + search areas. + if (tree) { //we need to ensure that any currently displayed nodes that get selected diff --git a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.html b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.html index 3d2772abcc..22aaa9f27d 100644 --- a/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.html +++ b/src/Umbraco.Web.UI.Client/src/views/common/dialogs/treepicker.html @@ -14,26 +14,6 @@