Don't use `confirm` in a digest cycle

Close #2097

Don't use `confirm` in a digest cycle
Close #2097
dde3d773 · Randall Leeds · f96a7996 · dde3d773
Commit dde3d773 authored May 12, 2015 by Randall Leeds
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

annotation.coffee h/static/scripts/directive/annotation.coffee +5 -2

No files found.
--- a/h/static/scripts/directive/annotation.coffee
+++ b/h/static/scripts/directive/annotation.coffee
@@ -103,8 +103,11 @@ AnnotationController = [
    # @description Deletes the annotation.
    ###
    this.delete = ->
-      if confirm "Are you sure you want to delete this annotation?"
-        annotationMapper.deleteAnnotation model
+      $timeout ->  # Don't use confirm inside the digest cycle
+        if confirm "Are you sure you want to delete this annotation?"
+          $scope.$apply ->
+            annotationMapper.deleteAnnotation model
+      , true

    ###*
    # @ngdoc method