Added ajaxContentAdded event that fires after content is added to DO…

…M. + fixed issue that could prevent modifying of responseText
dl988 · Jun 4, 2013 · efe8cec · efe8cec
1 parent 90bb326
commit efe8cec
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 7 deletions.
diff --git a/dist/jquery.magnific-popup.js b/dist/jquery.magnific-popup.js
@@ -15,6 +15,7 @@
  * Private static constants
  */
 var CLOSE_EVENT = 'Close',
+	AFTER_CLOSE_EVENT = 'AfterClose',
 	BEFORE_APPEND_EVENT = 'BeforeAppend',
 	MARKUP_PARSE_EVENT = 'MarkupParse',
 	OPEN_EVENT = 'Open',
@@ -446,6 +447,8 @@ MagnificPopup.prototype = {
 		mfp.content = null;
 		mfp.currTemplate = null;
 		mfp.prevHeight = 0;
+
+		_mfpTrigger(AFTER_CLOSE_EVENT);
 	},
 
 	updateSize: function(winHeight) {
@@ -1020,9 +1023,11 @@ $.magnificPopup.registerModule(AJAX_NS, {
 				url: item.src,
 				success: function(data, textStatus, jqXHR) {
 
-					_mfpTrigger('ParseAjax', jqXHR);
+					var data = {responseText:jqXHR.responseText};
+
+					_mfpTrigger('ParseAjax', data);
 
-					mfp.appendContent( $(jqXHR.responseText), AJAX_NS );
+					mfp.appendContent( $(data.responseText), AJAX_NS );
 
 					item.finished = true;
 
@@ -1035,7 +1040,8 @@ $.magnificPopup.registerModule(AJAX_NS, {
 					}, 16);
 
 					mfp.updateStatus('ready');
-
+
+					_mfpTrigger('AjaxContentAdded');
 				},
 				error: function() {
 					_removeAjaxCursor();