fix(api): Add test before removing controls

Closes #30
bcabanes · Mar 15, 2016 · fad7442 · fad7442
1 parent f5f6fb4
commit fad7442
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/dev/app/app.js b/dev/app/app.js
@@ -14,7 +14,7 @@ angular
 
     // Some cropper options.
     vm.imageUrl = 'assets/images/unsplash_' + getRandomInt(1, 1) + '.jpg';
-    vm.showControls = true;
+    vm.showControls = false;
     vm.fit = false;
 
     vm.myButtonLabels = {
@@ -32,12 +32,13 @@ angular
     };
 
     // Cropper API available when image is ready.
-    //vm.cropperApi = functigit staon(cropperApi) {
-      //cropperApi.zoomOut(10);
-      //cropperApi.zoomIn(20);
-      //cropperApi.rotate(270);
-      //cropperApi.fit();
+    //vm.cropperApi = function(cropperApi) {
+    //  cropperApi.zoomOut(10);
+    //  cropperApi.zoomIn(20);
+    //  cropperApi.rotate(270);
+    //  cropperApi.fit();
     //  vm.resultImage = cropperApi.crop();
+    //  //cropperApi.remove();
     //  $scope.$apply(); // Apply the changes.
     //};
 

diff --git a/src/imageCropper.js b/src/imageCropper.js
@@ -219,7 +219,7 @@ Cropper.prototype.buildDOM = function() {
 Cropper.prototype.remove = function() {
   var elements = this.elements;
   elements.target.removeChild(elements.wrapper);
-  elements.target.removeChild(elements.controls.wrapper);
+  if (this.options.showControls) elements.target.removeChild(elements.controls.wrapper);
 };
 
 Cropper.prototype.loadImage = function() {