From e4e4d1f9c099de6fb4b03a02205af905a3feb8b3 Mon Sep 17 00:00:00 2001 From: Heiko Kernbach Date: Thu, 21 Aug 2014 16:47:20 +0200 Subject: [PATCH] fixed wrong event binding to whole table rows instead of tbody only --- .../system/aardvark/frontend/js/views/documentsView.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/js/apps/system/aardvark/frontend/js/views/documentsView.js b/js/apps/system/aardvark/frontend/js/views/documentsView.js index 81524d1b0f..23b02b4df2 100644 --- a/js/apps/system/aardvark/frontend/js/views/documentsView.js +++ b/js/apps/system/aardvark/frontend/js/views/documentsView.js @@ -64,7 +64,7 @@ "click #filterSend" : "sendFilter", "click #addFilterItem" : "addFilterItem", "click .removeFilterItem" : "removeFilterItem", - "click #documentsTableID tr" : "clicked", + "click #documentsTableID tbody tr" : "clicked", "click #deleteDoc" : "remove", "click #deleteSelected" : "deleteSelectedDocs", "click #addDocumentButton" : "addDocument", @@ -287,14 +287,14 @@ changeEditMode: function (enable) { if (enable === false || this.editMode === true) { - $('#documentsTableID tr').css('cursor', 'default'); + $('#documentsTableID tbody tr').css('cursor', 'default'); $('.deleteButton').fadeIn(); $('.addButton').fadeIn(); $('.selected-row').removeClass('selected-row'); this.editMode = false; } else { - $('#documentsTableID tr').css('cursor', 'copy'); + $('#documentsTableID tbody tr').css('cursor', 'copy'); $('.deleteButton').fadeOut(); $('.addButton').fadeOut(); $('.selectedCount').text(0); @@ -578,7 +578,7 @@ getSelectedDocs: function() { var toDelete = []; - _.each($('#documentsTableID tr'), function(element) { + _.each($('#documentsTableID tbody tr'), function(element) { if ($(element).hasClass('selected-row')) { toDelete.push($($(element).children()[1]).find('.key').text()); }