Bug #35 when destroying it removes all click events for the document

Author: Tim-Intellipharm

lib/scripts/directive.js

@@ -36,12 +36,14 @@
                     }
                     $scope.initConfig();
 
-                    $document.on('click', function (evt) {
-                        if ($scope.find(evt.target).length === 0) {
-                            $scope.log('Color Picker: Document Click Event');
-                            $scope.hide();
-                        }
-                    });
+                    $document.on('click', $scope.onClick);
+                };
+
+                $scope.onClick = function(event) {
+                    if ($scope.find(event.target).length === 0) {
+                        $scope.log('Color Picker: Document Click Event');
+                        $scope.hide();
+                    }
                 };
 
                 $scope.initConfig = function() {
@@ -418,7 +420,7 @@
                 $scope.init();
 
                 $scope.$on('$destroy', function() {
-                    $document.off('click');
+                    $document.off('click', $scope.onClick);
                 });
             }
         };