From a5116cd8cf349ae85986e6362e54b3d86f41c241 Mon Sep 17 00:00:00 2001 From: Madhura Jayaratne Date: Thu, 17 Mar 2011 19:59:22 +0530 Subject: [PATCH] Proper handling of marked and hover classes for inline edit. --- js/sql.js | 27 ++++++++++++++++----------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/js/sql.js b/js/sql.js index f673b6b83f..acd9c8d541 100644 --- a/js/sql.js +++ b/js/sql.js @@ -474,16 +474,13 @@ $(document).ready(function() { $chg_submit.empty(); $chg_submit.append(data_vt); - var $this_row = $(this).parent().parent(); - if(parseInt(pos) % 2 == 0) { - $this_row.siblings("tr:eq(3) td:eq(" + pos + ")").removeClass("odd row_" + pos + " vpointer vmarker inline_edit_active").addClass("odd row_" + pos + " vpointer vmarker inline_edit_anchor"); + var $this_row = $(this).parents('tr'); + // changing inline_edit_active to inline_edit_anchor + $this_row.siblings("tr:eq(3) td:eq(" + pos + ")").removeClass("inline_edit_active").addClass("inline_edit_anchor"); - $this_row.siblings("tr:eq(3) td:eq(" + pos + ")").removeClass("odd row_" + pos + " vpointer vmarker inline_edit_active hover").addClass("odd row_" + pos + " vpointer vmarker inline_edit_anchor"); - } else { - $this_row.siblings("tr:eq(3) td:eq(" + pos + ")").removeClass("even row_" + pos + " vpointer vmarker inline_edit_active").addClass("even row_" + pos + " vpointer vmarker inline_edit_anchor"); + // removing marked and hover classes. + $this_row.parent('tbody').find('tr').find("td:eq(" + pos + ")").removeClass("marked hover"); - $this_row.siblings("tr:eq(3) td:eq(" + pos + ")").removeClass("even row_" + pos + " vpointer vmarker inline_edit_active hover").addClass("even row_" + pos + " vpointer vmarker inline_edit_anchor"); - } for( var i = 6; i <= rows + 2; i++){ if( $this_row.siblings("tr:eq(" + i + ") td:eq(" + pos + ")").hasClass("inline_edit") == false) { continue; @@ -899,9 +896,17 @@ $(document).ready(function() { } PMA_ajaxShowMessage(data.message); - $this_td.removeClass('inline_edit_active hover').addClass('inline_edit_anchor'); - $this_td.parent().removeClass('hover noclick'); - $this_td.siblings().removeClass('hover'); + + // changing inline_edit_active to inline_edit_anchor + $this_td.removeClass('inline_edit_active').addClass('inline_edit_anchor'); + + // removing hover, marked and noclick classes + $this_td.parent('tr').removeClass('noclick'); + if(disp_mode != 'vertical') { + $this_td.parent('tr').removeClass('hover').find('td').removeClass('hover'); + } else { + $this_td.parents('tbody').find('tr').find('td:eq(' + $this_td.index() + ')').removeClass('marked'); + } $input_siblings.each(function() { // Inline edit post has been successful. -- 2.11.4.GIT