From 4d0d3d3ad49b78ed8dd255b67d12f131b996a3a3 Mon Sep 17 00:00:00 2001
From: Brian Talbot <btalbot@edx.org>
Date: Mon, 8 Apr 2013 11:35:27 -0400
Subject: [PATCH] studio - alerts: refactored some alert and notification
 show/hide js added back to base.js

---
 cms/static/js/base.js | 25 ++++++++++++++-----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/cms/static/js/base.js b/cms/static/js/base.js
index 3186f7592b..bb3fe88510 100644
--- a/cms/static/js/base.js
+++ b/cms/static/js/base.js
@@ -48,17 +48,9 @@ $(document).ready(function () {
         (e).preventDefault();
     });
 
-    // alerts - manual close
-    $('.action-alert-close, .alert.has-actions .nav-actions a').click(function(e) {
-        (e).preventDefault();
-        $(this).closest('.wrapper-alert').removeClass('is-shown');
-    });
-
-    // notifications - manual & action-based close
-    $('.action-notification-close').click(function(e) {
-        (e).preventDefault();
-        $(this).closest('.wrapper-notification').removeClass('is-shown').addClass('is-hiding');
-    });
+    // alerts/notifications - manual close
+    $('.action-alert-close, .alert.has-actions .nav-actions a').bind('click', hideAlert);
+    $('.action-notification-close').bind('click', hideNotification);
 
     // nav - dropdown related
     $body.click(function (e) {
@@ -550,6 +542,17 @@ function removeDateSetter(e) {
     $block.find('.time').val('');
 }
 
+
+function hideNotification(e) {
+    (e).preventDefault();
+    $(this).closest('.wrapper-notification').removeClass('is-shown').addClass('is-hiding');
+}
+
+function hideAlert(e) {
+    (e).preventDefault();
+    $(this).closest('.wrapper-alert').removeClass('is-shown');
+}
+
 function showToastMessage(message, $button, lifespan) {
     var $toast = $('<div class="toast-notification"></div>');
     var $closeBtn = $('<a href="#" class="close-button">×</a>');