From 041f5f51a54b38d04403ac49fb7b9eb5b87f1d04 Mon Sep 17 00:00:00 2001 From: Loz Calver Date: Thu, 4 Jul 2013 15:57:35 +0100 Subject: [PATCH] FIX: UploadField action buttons aren't disabled when editing an item Toggle disabled classes/attributes based on form visibility instead of .toggle() Use jQuery.attr() simply because it looks nicer --- javascript/UploadField.js | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/javascript/UploadField.js b/javascript/UploadField.js index 1d6715732..a824deb8c 100644 --- a/javascript/UploadField.js +++ b/javascript/UploadField.js @@ -383,6 +383,7 @@ $( 'div.ss-upload:not(.disabled):not(.readonly) .ss-uploadfield-item-edit').entwine({ onclick: function(e) { var editform = this.closest('.ss-uploadfield-item').find('.ss-uploadfield-item-editform'); + var itemInfo = editform.prev('.ss-uploadfield-item-info'); var disabled; var iframe = editform.find('iframe'); @@ -406,8 +407,15 @@ disabled=this.find('ss-uploadfield-item-edit').siblings(); } editform.parent('.ss-uploadfield-item').removeClass('ui-state-warning'); - disabled.toggleClass('ui-state-disabled'); editform.toggleEditForm(); + + if (itemInfo.find('.toggle-details-icon').hasClass('opened')) { + disabled.addClass('ui-state-disabled'); + disabled.attr('disabled', 'disabled'); + } else { + disabled.removeClass('ui-state-disabled'); + disabled.removeAttr('disabled'); + } } e.preventDefault(); // Avoid a form submit return false;