Merge pull request #292 from kstenschke/master

media browsing: made previous/next media button wrap-around (continui…
ionize · Aug 20, 2015 · c7e415d · c7e415d
2 parents 3bd2acd + 8c26dfe
commit c7e415d
Showing 1 changed file with 55 additions and 11 deletions.
diff --git a/themes/admin/views/media/edit.php b/themes/admin/views/media/edit.php
@@ -368,19 +368,49 @@
 	function getTabContextIdentifier() {
 		return $('articleTabContent') === null ? 'page' : 'article';
 	}
+
 	function closeEditPopup(mediaID) {
 		$('wmedia' + mediaID + '_controls_button2').fireEvent('click');
 	}
+
+	function findFirstMediaID(mediaID) {
+		var previousID = mediaID;
+		while(previousID != null) {
+			previousID = findPreviousMediaID(mediaID);
+
+			if(previousID != null) {
+				mediaID = previousID;
+			}
+		}
+
+		return mediaID;
+	}
+
+	function findLastMediaID(mediaID) {
+		var lastID = mediaID;
+		while(lastID != null) {
+			lastID = findNextMediaID(mediaID);
+
+			if(lastID != null) {
+				mediaID = lastID;
+			}
+		}
+
+		return mediaID;
+	}
+
 	function findPreviousMediaID(mediaID) {
 		var elCurrent = $$('#' + getTabContextIdentifier() + 'TabContent .picture[data-id="' + mediaID + '"]')[0];
 		var elPrevious = $(elCurrent).getPrevious();
 		return elPrevious ? elPrevious.get('data-id') : null;
 	}
+
 	function findNextMediaID(mediaID) {
 		var elCurrent = $$('#' + getTabContextIdentifier() + 'TabContent .picture[data-id="' + mediaID + '"]')[0];
 		var elNext = $(elCurrent).getNext();
 		return elNext ? elNext.get('data-id') : null;
 	}
+
 	function editMediaByID(mediaID) {
 		if( mediaID != null ) {
 			var anchors = $$('#' + getTabContextIdentifier() + 'TabContent .picture[data-id="' + mediaID + '"] a.edit');
@@ -389,24 +419,38 @@ function editMediaByID(mediaID) {
 			}
 		}
 	}
+
 	$('imagePreviousLink').addEvent('click', function () {
-		var clickedID = parseInt( $('imagePreviousLink').get('data-id'), 10);
-		closeEditPopup(clickedID);
-		editMediaByID( findPreviousMediaID(clickedID) );
+		var clickedMediaID = parseInt( $('imagePreviousLink').get('data-id'), 10);
+		closeEditPopup(clickedMediaID);
+
+		var previousMediaID = findPreviousMediaID(clickedMediaID);
+		if(previousMediaID != null) {
+			editMediaByID(previousMediaID);
+		} else {
+			editMediaByID(findLastMediaID(clickedMediaID));
+		}
 	});
+
 	$('imageNextLink').addEvent('click', function () {
-		var clickedID = parseInt( $('imagePreviousLink').get('data-id'), 10);
-		closeEditPopup(clickedID);
-		editMediaByID( findNextMediaID(clickedID) );
+		var clickedMediaID = parseInt( $('imagePreviousLink').get('data-id'), 10);
+		closeEditPopup(clickedMediaID);
+
+		var nextMediaID = findNextMediaID(clickedMediaID);
+		if(nextMediaID != null) {
+			editMediaByID(nextMediaID);
+		} else {
+			editMediaByID(findFirstMediaID(clickedMediaID))
+		}
 	});
 
 	// Extend Fields
 	var mediaExtendManager<?php echo $id_media; ?> = new ION.ExtendManager({
-		parent: 'media',
-		id_parent: id_media,
-		destination: 'mediaTab' + id_media,
-		destinationTitle: Lang.get('ionize_title_extend_fields'),
-		onLoaded: function(extendManager)
+		parent				: 'media',
+		id_parent			: id_media,
+		destination			: 'mediaTab' + id_media,
+		destinationTitle	: Lang.get('ionize_title_extend_fields'),
+		onLoaded			: function(extendManager)
 		{
 			extendManager.getParentInstances();
 		}