Merge pull request #13834 from MrPetovan/task/remove-delete-rotator

[frio] Move item deletion rotator to button

view/js/main.js

@@ -379,7 +379,7 @@ $(function() {
 	// Allow folks to stop the ajax page updates with the pause/break key
 	$(document).keydown(function(event) {
 		// Pause/Break or Ctrl + Space
-		if (event.which === 19 || (!event.shiftKey && !event.altKey && event.ctrlKey && event.which === 32)) {
+		if (event.which === 19 || (!event.metaKey && !event.shiftKey && !event.altKey && event.ctrlKey && event.which === 32)) {
 			event.preventDefault();
 			if (stopped === false) {
 				stopped = true;

view/theme/frio/templates/jot-header.tpl

@@ -146,44 +146,41 @@
 
 	function deleteCheckedItems() {
 		if (confirm('{{$delitems}}')) {
-			var checkedstr = '';
-			var ItemsToDelete = {};
+			let checkedstr = '';
+			const ItemsToDelete = {};
 
-			$("#item-delete-selected").hide();
-			$('#item-delete-selected-rotator').show();
-			$('body').css('cursor', 'wait');
+			$('#item-delete-selected').prop('disabled', true);
+			$('#item-delete-selected i').toggleClass('fa-trash fa-hourglass fa-spin');
 
 			$('.item-select').each(function () {
 				if ($(this).is(':checked')) {
-					if(checkedstr.length != 0) {
+					if (checkedstr.length > 0) {
 						checkedstr = checkedstr + ',' + $(this).val();
-						var deleteItem = this.closest(".wall-item-container");
-						ItemsToDelete[deleteItem.id] = deleteItem;
-					}
-					else {
+					} else {
 						checkedstr = $(this).val();
 					}
 
 					// Get the corresponding item container
-					var deleteItem = this.closest(".wall-item-container");
+					const deleteItem = this.closest(".wall-item-container");
 					ItemsToDelete[deleteItem.id] = deleteItem;
 				}
 			});
 
 			// Fade the container from the items we want to delete
-			for(var key in  ItemsToDelete) {
+			for (const key in ItemsToDelete) {
 				$(ItemsToDelete[key]).fadeTo('fast', 0.33);
-			};
+			}
 
 			$.post('item', {dropitems: checkedstr}, function (data) {
 			}).done(function () {
 				// Loop through the ItemsToDelete Object and remove
 				// corresponding item div
-				for(var key in  ItemsToDelete) {
+				for (const key in ItemsToDelete) {
 					$(ItemsToDelete[key]).remove();
 				}
-				$('body').css('cursor', 'auto');
-				$('#item-delete-selected-rotator').hide();
+
+				$('#item-delete-selected i').toggleClass('fa-trash fa-hourglass fa-spin')
+				$('#item-delete-selected').prop('disabled', false).hide();
 			});
 		}
 	}

view/theme/frio/templates/threaded_conversation.tpl

@@ -20,6 +20,5 @@
 <button type="button" id="item-delete-selected" class="btn btn-link" title="{{$dropping}}" onclick="deleteCheckedItems();" data-toggle="tooltip">
 	<i class="fa fa-trash" aria-hidden="true"></i>
 </button>
-<img id="item-delete-selected-rotator" class="like-rotator" src="images/rotator.gif" style="display: none;" />
 {{/if}}
 {{/if}}