Merge pull request #423 from /issues/343/1

Fixes #343 - Use flash:error event in issue model.
webcompat · Nov 25, 2014 · 21680ef · 21680ef
2 parents 84d3087 + e465834
commit 21680ef
Showing 1 changed file with 4 additions and 18 deletions.
diff --git a/webcompat/static/js/lib/models/issue.js b/webcompat/static/js/lib/models/issue.js
@@ -68,15 +68,8 @@
         }
       },
       error: function() {
-        $('<div></div>', {
-          'class': 'flash error',
-          'text': 'There was an error editing this issues\'s status.'
-        }).appendTo('body');
-
-        setTimeout(function(){
-          var __flashmsg = $('.flash');
-          if (__flashmsg.length) {__flashmsg.fadeOut();}
-        }, 2000);
+        var msg = 'There was an error editing this issues\'s status.';
+        wcEvents.trigger('flash:error', {message: msg, timeout: 2000});
       }
     });
   },
@@ -102,15 +95,8 @@
         self.set('labels', response);
       },
       error: function() {
-        $('<div></div>', {
-          'class': 'flash error',
-          'text': 'There was an error setting labels.'
-        }).appendTo('body');
-
-        setTimeout(function(){
-          var __flashmsg = $('.flash');
-          if (__flashmsg.length) {__flashmsg.fadeOut();}
-        }, 2000);
+        var msg = 'There was an error setting labels.';
+        wcEvents.trigger('flash:error', {message: msg, timeout: 2000});
       }
     });
   }